Change overly strict Assert in TransactionGroupUpdateXidStatus.
authorAmit Kapila <akapila@postgresql.org>
Thu, 12 Dec 2019 06:21:30 +0000 (11:51 +0530)
committerAmit Kapila <akapila@postgresql.org>
Tue, 17 Dec 2019 03:59:22 +0000 (09:29 +0530)
commitaf3290f5e790dcd1be3ac209be1805626f4ebac8
tree05c2f6b6a4167d0772cb9c4b110c14e76e5cd622
parentfcf3b6917bd8f6f9f463e3e42e53d6ff9612e327
Change overly strict Assert in TransactionGroupUpdateXidStatus.

This Assert thought that an overflowed transaction can never get registered
for the group update.  But that is not true, because even when the number
of children for a transaction got reduced, the overflow flag is not
changed.  And, for group update, we only care about the current number of
children for a transaction that is being committed.

Based on comments by Andres Freund, remove a redundant Assert in
TransactionIdSetPageStatus as we already had a static Assert for the same
condition a few lines earlier.

Reported-by: Vignesh C
Author: Dilip Kumar
Reviewed-by: Amit Kapila
Backpatch-through: 11
Discussion: https://postgr.es/m/CAFiTN-s5=uJw-Z6JC9gcqtBSjXsrHnU63PXBrA=pnBjqnkm5UA@mail.gmail.com
src/backend/access/transam/clog.c