lp://staging/~cjwatson/storm/refactor-exception-wrapping

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://staging/storm at revision 509

Simon Poirier (community): Approve on 2019-07-02

Diff: 423 lines (+228/-31)

6 files modified

NEWS (+4/-0)
storm/database.py (+132/-1)
storm/databases/postgres.py (+28/-7)
storm/databases/sqlite.py (+15/-9)
storm/exceptions.py (+46/-14)
tests/database.py (+3/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1006711: Use ABCMeta for storm.exceptions?	Undecided	Fix Released
Bug #1170063: Storm crashes on recent psycopg2	Undecided	Fix Released

Link a bug report

Related blueprints

512. By Colin Watson on 2019-07-02

Use more neutral language.

511. By Colin Watson on 2019-07-02

Use superclass __setattr__ rather than changing __dict__ directly.

510. By Colin Watson on 2019-06-26

Improve comment.

509. By Colin Watson on 2019-06-26

Make wrapper exceptions look slightly more like the DB-API originals.

508. By Colin Watson on 2019-06-25

Wrap DB-API exceptions rather than injecting virtual base classes.

Re-raise DB-API exceptions wrapped in exception types that inherit from
both StormError and the appropriate DB-API exception type, rather than
injecting virtual base classes. This preserves existing exception
handling in applications while also being a viable approach in Python 3.

507. By Colin Watson on 2019-06-07

Import from six.moves where appropriate. [r=simpoir]

506. By Thiago Bellini on 2019-06-07

Use Python 3-compatible exec functions. [r=cjwatson]

505. By Martin v. Löwis on 2019-06-07

Make cextensions module build with 3.x. [r=cjwatson]

504. By Thiago Bellini on 2019-06-07

Use sys.maxsize rather than sys.maxint. [r=cjwatson]

503. By Colin Watson on 2019-06-06

Fix reversed logic for (no)cextensions factors in tox.ini. [r=simpoir]