diff options
Diffstat (limited to 'src/backend/access/transam/transam.c')
| -rw-r--r-- | src/backend/access/transam/transam.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/backend/access/transam/transam.c b/src/backend/access/transam/transam.c index 3a28dcc43aa..7629904bbf7 100644 --- a/src/backend/access/transam/transam.c +++ b/src/backend/access/transam/transam.c @@ -110,7 +110,8 @@ TransactionLogFetch(TransactionId transactionId) * transaction tree. * * See also TransactionIdIsInProgress, which once was in this module - * but now lives in procarray.c. + * but now lives in procarray.c, as well as comments at the top of + * heapam_visibility.c that explain how everything fits together. * ---------------------------------------------------------------- */ @@ -176,6 +177,12 @@ TransactionIdDidCommit(TransactionId transactionId) * * Note: * Assumes transaction identifier is valid and exists in clog. + * + * Returns true only for explicitly aborted transactions, as transactions + * implicitly aborted due to a crash will commonly still appear to be + * in-progress in the clog. Most of the time TransactionIdDidCommit(), + * with a preceding TransactionIdIsInProgress() check, should be used + * instead of TransactionIdDidAbort(). */ bool /* true if given transaction aborted */ TransactionIdDidAbort(TransactionId transactionId) |