Do not report txn commit/abort to the GTM via parallel worker.
authorPavan Deolasee <pavan.deolasee@gmail.com>
Thu, 21 Feb 2019 13:20:18 +0000 (18:50 +0530)
committerPavan Deolasee <pavan.deolasee@gmail.com>
Thu, 21 Feb 2019 13:34:39 +0000 (19:04 +0530)
commitce5581bd0f071b994299a58db3c3e6e137a226c4
tree4440ab2695b6b11f66ffb6c87fea9d866ac46277
parent024bce5f5e45cb8e80782a80bdf53ef425254d1d
Do not report txn commit/abort to the GTM via parallel worker.

We've seen a race condition where a parallel worker process on the coordinator
pre-maturely reports txn status to the GTM, resulting in cleanup of the txn
from the GTM. This is wrong and we must let the leader process do this.

This may not fully explain some of the recent reports we have seen with
catalog corruptions where an updated row is marked visible as its xmax is seen
as aborted. But this looks like a sane fix anyways. So push it while we
continue to investigate the reported problem.
src/backend/access/transam/xact.c