Add missing break out seqscan loop in logical replication
authorAlvaro Herrera
Mon, 3 Feb 2020 21:59:12 +0000 (18:59 -0300)
committerAlvaro Herrera
Mon, 3 Feb 2020 21:59:12 +0000 (18:59 -0300)
commitbeefab6540deca589c2d9a1d0f91fd98f11d303a
tree87119cad05f5b545bc4c5f88a132f7a6bcebb8b8
parentea7857dddb54c371f70be7215ea3b72cb70265bf
Add missing break out seqscan loop in logical replication

When replica identity is FULL (an admittedly unusual case), the loop
that searches for tuples in execReplication.c didn't stop scanning the
table when once a matching tuple was found.  Add the missing 'break'.

Note slight behavior change: we now return the first matching tuple
rather than the last one.  They are supposed to be indistinguishable
anyway, so this shouldn't matter.

Author: Konstantin Knizhnik
Discussion: https://postgr.es/m/379743f6-ae91-b866-f7a2-5624e6d2b0a4@postgrespro.ru
src/backend/executor/execReplication.c