From 2336f842843e5bd26779692b39eaf0ef9d4d31da Mon Sep 17 00:00:00 2001 From: Alvaro Herrera Date: Sat, 12 Aug 2017 21:36:07 -0400 Subject: [PATCH] Reword comment for clarity Reported by Masahiko Sawada Discussion: https://postgr.es/m/CAD21AoB+ycZ2z-4Ye=6MfQ_r0aV5r6cvVPw4kOyPdp6bHqQoBQ@mail.gmail.com --- src/backend/replication/logical/snapbuild.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/backend/replication/logical/snapbuild.c b/src/backend/replication/logical/snapbuild.c index 153d1712ef..0ca4fa5d25 100644 --- a/src/backend/replication/logical/snapbuild.c +++ b/src/backend/replication/logical/snapbuild.c @@ -1116,9 +1116,9 @@ SnapBuildProcessRunningXacts(SnapBuild *builder, XLogRecPtr lsn, xl_running_xact * so, because we only need to do it for catalog transactions since we * only ever look at those. * - * NB: Because of that xmax can be lower than xmin, because we only - * increase xmax when a catalog modifying transaction commits. While odd - * looking, it's correct and actually more efficient this way since we hit + * NB: We only increase xmax when a catalog modifying transaction commits + * (see SnapBuildCommitTxn). Because of this, xmax can be lower than xmin, + * which looks odd but is correct and actually more efficient, since we hit * fast paths in tqual.c. */ builder->xmin = running->oldestRunningXid;