Re: BUG #14228: replication slot catalog_xmin not cleared on slot reuse

* Michael Paquier (michael.paquier@gmail.com) wrote:
> On Wed, Jul 6, 2016 at 12:56 PM, Andrew Gierth
> <andrew@tao11.riddles.org.uk> wrote:
> >>>>>> "Michael" =3D=3D Michael Paquier <michael.paquier@gmail.com> write=
s:
> >
> >  >> When creating a physical replication slot, the catalog_xmin field of
> >  >> the new slot is not initialized. If the slot storage had previously
> >  >> been used for a logical slot, the old catalog_xmin will remain in
> >  >> place and interfere with vacuum.
> >
> >  Michael> Good catch! The same applies to confirmed_flush_lsn, which is
> >  Michael> used only by logical decoding and should remain as NULL for
> >  Michael> physical slots.  So I propose the patch attached to address
> >  Michael> both problems.
> >
> > What about slot->effective_catalog_xmin ?
>=20
> Yes. I guess so, as well as the other candidate_* fields in the slot
> to begin from a clean state.

Seems like we should try to get this in before the next round of point
releases...?

Thanks!

Stephen