From 5ee940e1cdb6af3af52bb01e44aac63f3a73a28d Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 15 Apr 2018 20:23:59 -0400 Subject: [PATCH] Add temporary debugging assertion, in 9.4 branch only. Buildfarm member okapi has been failing the multiple-cic isolation test for months now, but only in 9.4. To narrow down the possible causes, add an Assert testing that CREATE INDEX CONCURRENTLY is advertising zero xmin before waiting for other transactions to end. I'm not sure that this would hold in general, so this assertion isn't meant to get released, but it passes all 9.4 regression tests for me. Will revert once we see how okapi responds. --- src/backend/commands/indexcmds.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/backend/commands/indexcmds.c b/src/backend/commands/indexcmds.c index 05afcb0088d..9d48032617c 100644 --- a/src/backend/commands/indexcmds.c +++ b/src/backend/commands/indexcmds.c @@ -773,6 +773,9 @@ DefineIndex(Oid relationId, UnregisterSnapshot(snapshot); InvalidateCatalogSnapshot(); + /* Temporary debug check: we should be advertising we have no snapshots. */ + Assert(MyPgXact->xmin == InvalidTransactionId); + /* * The index is now valid in the sense that it contains all currently * interesting tuples. But since it might not contain tuples deleted just -- 2.39.5