On 2020-Jun-02, Michael Paquier wrote:
> I can note as well that StartLogicalReplication() moves in this sense
> by setting xlogreader to be the one from logical_decoding_ctx once the
> decoding context has been created.
>
> This results in the attached. The extra test from upthread to check
> that logical decoding is not allowed in a non-database WAL sender is a
> good idea, so I have kept it.
I don't particularly disagree with your proposed patch -- in fact, it
seems to make things cleaner. It is a little wasteful, but I don't
really mind that. It's just some memory, and it's not a significant
amount.
That said, I would *also* apply Kyotaro's proposed patch to prohibit a
physical standby running with a logical slot, if only because that
reduces the number of combinations that we need to test and keep our
collective heads straight about. Just reject the weird case and have
one type of slot for each type of replication. I didn't even think this
was at all possible.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services