diff options
author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-07-29 17:09:06 -0400 |
---|---|---|
committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-07-29 17:09:06 -0400 |
commit | ce197e91d02ca38ac1b956a3e0c11596099e7893 (patch) | |
tree | 994d87ddf47e7f421c74c63e3dbaf0555ad5c6be /src/backend/access/transam/xlog.c | |
parent | fcf4c0ae1db8594cf3d3e84d2564616497eecd5a (diff) | |
download | postgresql-ce197e91d02ca38ac1b956a3e0c11596099e7893.tar.gz postgresql-ce197e91d02ca38ac1b956a3e0c11596099e7893.zip |
Close yet another race condition in replication slot test code
Buildfarm shows that this test has a further failure mode when a
checkpoint starts earlier than expected, so we detect a "checkpoint
completed" line that's not the one we want. Change the config to try
and prevent this.
Per buildfarm
While at it, update one comment that was forgotten in commit
d18e75664a2f.
Author: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Discussion: https://postgr.es/m/20210729.162038.534808353849568395.horikyota.ntt@gmail.com
Diffstat (limited to 'src/backend/access/transam/xlog.c')
-rw-r--r-- | src/backend/access/transam/xlog.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c index 3479402272b..e717930ecf2 100644 --- a/src/backend/access/transam/xlog.c +++ b/src/backend/access/transam/xlog.c @@ -9329,7 +9329,7 @@ CreateCheckPoint(int flags) if (!RecoveryInProgress()) TruncateSUBTRANS(GetOldestTransactionIdConsideredRunning()); - /* Real work is done, but log and update stats before releasing lock. */ + /* Real work is done; log and update stats. */ LogCheckpointEnd(false); /* Reset the process title */ |