Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

Final support for transaction #11

Open
wants to merge 119 commits into
base: master
Choose a base branch
from
Open

Conversation

xadhoom
Copy link
Contributor

@xadhoom xadhoom commented Jun 20, 2011

Hi,

as discussed in the issues #9, here's the final support for trasactions, along with tests and docs.
Also in sync with current master branch.

Hope all is ok!

Brian Muller and others added 27 commits March 18, 2011 07:48
TODO:
* add transaction support for object deletion (and tests)
* add docs
Postgresql tests should work but cannot test right now.
TODO:
* support transaction on object delete (and add tests)
* add docs
Conflicts:
	.gitignore
	twistar/dbconfig/base.py
	twistar/tests/test_dbobject.py
Fixed Conflict:
	twistar/tests/test_utils.py
@xadhoom
Copy link
Contributor Author

xadhoom commented Jul 25, 2011

Hi,
I've updated to latest master revision. Any chance to see it merged :)
If something is wrong, I'll be glad to fix it. I think transaction are really useful in complex scenarios...

davec82 and others added 12 commits June 8, 2021 12:39
…tion-message

fix exception message when failing to commit
currently connection errors in the {start,commit,rollback}Transaction
methods will cause the underlying thread to never stop, causing the pool
to eventually exhaust all the resources

This fixes the issue by capturing the error, stopping the worker, and
propagating back the exception towards the caller
…topped

always stop txthreadworker, even after unexpected exceptions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants