diff options
| author | Andres Freund <andres@anarazel.de> | 2020-08-16 12:57:01 -0700 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2020-08-16 12:57:01 -0700 |
| commit | 49967da65aec970fcda123acc681f1df5d70bfc6 (patch) | |
| tree | 6cf20d5c6e151998fd7bbd3c392da6c1ea795ae0 /src/backend/access/heap/heapam.c | |
| parent | 676a9c3cc4b5f1d262c29de318868948513f0fa0 (diff) | |
| download | postgresql-49967da65aec970fcda123acc681f1df5d70bfc6.tar.gz postgresql-49967da65aec970fcda123acc681f1df5d70bfc6.zip | |
Make vacuum a bit more verbose to debug BF failure.
This is temporary. While possibly some more error checking / debugging
in this path would be a good thing, it'll not look exactly like this.
Discussion: https://postgr.es/m/20200816181604.l54m6kss5ntd6xow@alap3.anarazel.de
Diffstat (limited to 'src/backend/access/heap/heapam.c')
| -rw-r--r-- | src/backend/access/heap/heapam.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c index 9b5f417eac4..8eb276e4644 100644 --- a/src/backend/access/heap/heapam.c +++ b/src/backend/access/heap/heapam.c @@ -6048,7 +6048,16 @@ FreezeMultiXactId(MultiXactId multi, uint16 t_infomask, TransactionIdIsInProgress(members[i].xid)) { /* running locker cannot possibly be older than the cutoff */ - Assert(!TransactionIdPrecedes(members[i].xid, cutoff_xid)); + if (TransactionIdPrecedes(members[i].xid, cutoff_xid)) + { + /* temporary on-bf debugging */ + elog(PANIC, "too old alive locker: multi: %u, member xid: %u, memb-current: %d, memb-progress: %d, cutoff: %u, cutoff-multi: %u, relfrozenxid: %u, relminmxid: %u", + multi, members[i].xid, + TransactionIdIsCurrentTransactionId(members[i].xid), + TransactionIdIsInProgress(members[i].xid), + cutoff_xid, cutoff_multi, + relfrozenxid, relminmxid); + } newmembers[nnewmembers++] = members[i]; has_lockers = true; } |