Opened 9 years ago

Closed 6 years ago

#6096 closed (fixed)

commit_on_success does not handle SystemExit

Reported by: eric@… Owned by: Fabián Ezequiel Gallina
Component: Database layer (models, ORM) Version: master
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

I do not know if commit_on_success intentionally only catches Exception instead of BaseException (or just a plain except:). I was writing a command-line script that called sys.exit(1) when it detected a problem, but the transaction manager was raising TransactionManagementError("Transaction managed block ended with pending COMMIT/ROLLBACK") and it took me a long time to figure out what was going on.

If you don't want to change it to catch all exceptions, may I suggest adding a small note to the docs?

Change History (4)

comment:1 Changed 9 years ago by Chris Beaven

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedDesign decision needed

Yea, sounds like a bug... passing to core to check.

comment:2 Changed 6 years ago by Malcolm Tredinnick

Triage Stage: Design decision neededAccepted

Change this (you can use a bare "except" and just document it). I think this is just a consequence of us old-timers not always remembering SystemEerror got moved outside of Exception in the hierarchy at some point in the last decade. We have more things to remember than you young guys, so be nice to us. :-)

comment:3 Changed 6 years ago by Fabián Ezequiel Gallina

Owner: changed from nobody to Fabián Ezequiel Gallina

comment:4 Changed 6 years ago by Fabián Ezequiel Gallina

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.
Back to Top