From: | Arseny Sher <a(dot)sher(at)postgrespro(dot)ru> |
---|---|
To: | Masahiko Sawada <masahiko(dot)sawada(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Subject: | Re: logical copy_replication_slot issues |
Date: | 2020-02-10 14:01:25 |
Message-ID: | 87v9oelex6.fsf@ars-thinkpad |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Masahiko Sawada <masahiko(dot)sawada(at)2ndquadrant(dot)com> writes:
> I've attached the draft patch fixing this issue but I'll continue
> investigating it more deeply.
There also should be a check that source slot itself has consistent
snapshot (valid confirmed_flush) -- otherwise it might be possible to
create not initialized slot which is probably not an error, but weird
and somewhat meaningless. Paranoically, this ought to be checked in both
src slot lookups.
With this patch it seems like the only thing
create_logical_replication_slot does is ReplicationSlotCreate, which
questions the usefulness of this function. On the second look,
CreateInitDecodingContext checks plugin sanity (ensures it exists), so
probably it's fine.
-- cheers, arseny
From | Date | Subject | |
---|---|---|---|
Next Message | Dmitry Igrishin | 2020-02-10 14:33:17 | Is it memory leak or not? |
Previous Message | Craig Ringer | 2020-02-10 13:53:03 | Re: Postgres 32 bits client compilation fail. Win32 bits client is supported? |