From: Gunnlaugur Þór Briem Date: Thu, 21 Aug 2014 18:42:08 +0000 (+0000) Subject: Add note on begin_nested requiring rollback/commit X-Git-Tag: rel_0_9_8~51 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=dec550ccf6886284286297436feb96709a9c656d;p=thirdparty%2Fsqlalchemy%2Fsqlalchemy.git Add note on begin_nested requiring rollback/commit Avoid confusion about rollback/commit "must be issued" after ``session.begin_nested()`` --- this might be taken to mean call must be *added*, but that's only true if not using the return value as a context manager. --- diff --git a/doc/build/orm/session.rst b/doc/build/orm/session.rst index b47e70d537..78ae1ba814 100644 --- a/doc/build/orm/session.rst +++ b/doc/build/orm/session.rst @@ -1773,7 +1773,10 @@ method:: of times, which will issue a new SAVEPOINT with a unique identifier for each call. For each :meth:`~.Session.begin_nested` call, a corresponding :meth:`~.Session.rollback` or -:meth:`~.Session.commit` must be issued. +:meth:`~.Session.commit` must be issued. (But note that if the return value is +used as a context manager, i.e. in a with-statement, then this rollback/commit +is issued by the context manager upon exiting the context, and so should not be +added explicitly.) When :meth:`~.Session.begin_nested` is called, a :meth:`~.Session.flush` is unconditionally issued