Skip to content

Use dbapi PEP 249 errors - #204

Merged
ebyhr merged 3 commits into
trinodb:masterfrom
mdesmet:pep249-errors
Jul 20, 2022
Merged

Use dbapi PEP 249 errors#204
ebyhr merged 3 commits into
trinodb:masterfrom
mdesmet:pep249-errors

Conversation

@mdesmet

@mdesmet mdesmet commented Jul 19, 2022

Copy link
Copy Markdown
Contributor

Fixes #199

@cla-bot cla-bot Bot added the cla-signed label Jul 19, 2022
@mdesmet mdesmet changed the title Pep249 errors Use dbapi PEP 249 errors Jul 19, 2022
@mdesmet
mdesmet requested review from ebyhr and hashhar July 19, 2022 21:49

@hashhar hashhar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last commit was confusing to review - hopefully someday git will be smart enough

@ebyhr
ebyhr merged commit 748d197 into trinodb:master Jul 20, 2022
@ebyhr

ebyhr commented Jul 20, 2022

Copy link
Copy Markdown
Member

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

DB API raises exceptions which are not derived from the PEP 249 database API specification

3 participants