diff options
| author | Robert Haas <rhaas@postgresql.org> | 2018-07-16 17:33:22 -0400 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2018-07-16 17:33:22 -0400 |
| commit | 32df1c9afa5a11e37b154fe50df7a4f016f289e4 (patch) | |
| tree | d40e88f017cc178ea2b0cec6047255c565ed6818 /src/backend/access/transam/xact.c | |
| parent | f7cb2842bf47715133b40e4a503f35dbe60d1b72 (diff) | |
| download | postgresql-32df1c9afa5a11e37b154fe50df7a4f016f289e4.tar.gz postgresql-32df1c9afa5a11e37b154fe50df7a4f016f289e4.zip | |
Add subtransaction handling for table synchronization workers.
Since the old logic was completely unaware of subtransactions, a
change made in a subsequently-aborted subtransaction would still cause
workers to be stopped at toplevel transaction commit. Fix that by
managing a stack of worker lists rather than just one.
Amit Khandekar and Robert Haas
Discussion: http://postgr.es/m/CAJ3gD9eaG_mWqiOTA2LfAug-VRNn1hrhf50Xi1YroxL37QkZNg@mail.gmail.com
Diffstat (limited to 'src/backend/access/transam/xact.c')
| -rw-r--r-- | src/backend/access/transam/xact.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c index 1da1f13ef33..9aa63c8792b 100644 --- a/src/backend/access/transam/xact.c +++ b/src/backend/access/transam/xact.c @@ -4637,6 +4637,7 @@ CommitSubTransaction(void) AtEOSubXact_HashTables(true, s->nestingLevel); AtEOSubXact_PgStat(true, s->nestingLevel); AtSubCommit_Snapshot(s->nestingLevel); + AtEOSubXact_ApplyLauncher(true, s->nestingLevel); /* * We need to restore the upper transaction's read-only state, in case the @@ -4790,6 +4791,7 @@ AbortSubTransaction(void) AtEOSubXact_HashTables(false, s->nestingLevel); AtEOSubXact_PgStat(false, s->nestingLevel); AtSubAbort_Snapshot(s->nestingLevel); + AtEOSubXact_ApplyLauncher(false, s->nestingLevel); } /* |