_
On Wed, Jul 6, 2016 at 12:56 PM, Andrew Gierth
<andrew@tao11.riddles.org.uk> wrote:
>>>>>> "Michael" == Michael Paquier <michael.paquier@gmail.com> writes:
>
> >> 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 ?
Yes. I guess so, as well as the other candidate_* fields in the slot
to begin from a clean state.
--
Michael