pheasant icon indicating copy to clipboard operation
pheasant copied to clipboard

Support nested transactions using SAVEPOINT

Open Jud opened this issue 10 years ago • 4 comments

The base branch for #111 was deleted, so I'm opening a new PR with a new base branch.

(Fixes #56)

Jud avatar Apr 23 '15 14:04 Jud

That's quite a PR, nice work! I'll have a look at this once we have 1.3.2 released. Would you mind writing some documentation for this as well? IIRC transactions are currently not explained at all in the docs.

bjornpost avatar Apr 24 '15 19:04 bjornpost

Code LGTM. An example of how to use in the docs would be great. While you're at it, maybe we can also add an extra example of how Events work?

bjornpost avatar Aug 24 '15 11:08 bjornpost

@Jud ping for docs :-)

bjornpost avatar May 07 '16 17:05 bjornpost

@bjornpost re: the docs, they are relatively unchanged. IMO this just makes the existing transaction code work like a casual observer would expect them to. I'll open another issue for cleaning up event docs.

Edit: btw, we've been running this code in production since ~2015. May be some corner cases we're not running into, but generally works as expected.

Jud avatar Jan 05 '18 15:01 Jud