]> granicus.if.org Git - postgresql/commit
Fix thinko introduced in 2bef06d516460 et al.
authorAndres Freund <andres@anarazel.de>
Sun, 6 Aug 2017 03:52:53 +0000 (20:52 -0700)
committerAndres Freund <andres@anarazel.de>
Sun, 6 Aug 2017 21:21:22 +0000 (14:21 -0700)
commit869a5869e5ba7be0870c552e987158659d450e6c
treef73d37d9882562594b7e3f4f425f6f6dd303b13d
parent062291f228aca90b3e2cfda7a7e5024421ce466b
Fix thinko introduced in 2bef06d516460 et al.

The callers for GetOldestSafeDecodingTransactionId() all inverted the
argument for the argument introduced in 2bef06d516460. Luckily this
appears to be inconsequential for the moment, as we wait for
concurrent in-progress transaction when assembling a
snapshot. Additionally this could only make a difference when adding a
second logical slot, because only a pre-existing slot could cause an
issue by lowering the returned xid dangerously much.

Reported-By: Antonin Houska
Discussion: https://postgr.es/m/32704.1496993134@localhost
Backport: 9.4-, where 2bef06d516460 was backpatched to.
src/backend/replication/logical/logical.c
src/backend/replication/logical/snapbuild.c