mirror/postgresql
2
0
Fork 0
mirror of https://git.postgresql.org/git/postgresql.git synced 2025-03-01 19:45:33 +08:00
Code Issues Packages Projects Releases Wiki Activity

Doc: add comments about PreventInTransactionBlock/IsInTransactionBlock.

Browse Source 
Add a little to the header comments for these functions to make it
clearer what guarantees about commit behavior are provided to callers.
(See commit f92944137 for context.)

Although this is only a comment change, it's really documentation
aimed at authors of extensions, so it seems appropriate to back-patch.

Yugo Nagata and Tom Lane, per further discussion of bug #17434.

Discussion: https://postgr.es/m/17434-d9f7a064ce2a88a3@postgresql.org
This commit is contained in:
Tom Lane 2022-11-09 11:08:52 -05:00
parent b28ac1d24d
commit e613ace1f0
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/backend/access/transam/xact.c
View File

@ -3448,6 +3448,10 @@ AbortCurrentTransaction(void)
* a transaction block, typically because they have non-rollback-able
* side effects or do internal commits.
*
* If this routine completes successfully, then the calling statement is
* guaranteed that if it completes without error, its results will be
* committed immediately.
*
* If we have already started a transaction block, issue an error; also issue
* an error if we appear to be running inside a user-defined function (which
* could issue more commands and possibly cause a failure after the statement
@ -3573,6 +3577,10 @@ CheckTransactionBlock(bool isTopLevel, bool throwError, const char *stmtType)
* a transaction block than when running as single commands. ANALYZE is
* currently the only example.
*
* If this routine returns "false", then the calling statement is
* guaranteed that if it completes without error, its results will be
* committed immediately.
*
* isTopLevel: passed down from ProcessUtility to determine whether we are
* inside a function.
*/