database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-06-08 22:02:03 +03:00
Code

Add temporary debugging assertion, in 9.4 branch only.

Browse Source 
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.
This commit is contained in:
Tom Lane 2018-04-15 20:23:59 -04:00
parent e6b71727af
commit 5ee940e1cd
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/backend/commands/indexcmds.c
View File

@ -773,6 +773,9 @@ DefineIndex(Oid relationId,
UnregisterSnapshot(snapshot); UnregisterSnapshot(snapshot);
InvalidateCatalogSnapshot(); 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 * The index is now valid in the sense that it contains all currently
* interesting tuples. But since it might not contain tuples deleted just * interesting tuples. But since it might not contain tuples deleted just