On 6/25/17 06:35, Petr Jelinek wrote:
> - Do LockSharedObject in ALTER SUBSCRIPTION, DROP SUBSCRIPTION (this one
> is preexisting but mentioning it for context), SetSubscriptionRelState,
> AddSubscriptionRelState, and in the logicalrep_worker_launch. This means
> we use granular per object locks to deal with concurrency.
I have committed those locking changes, as we had already discussed them
previously. This should address the open item.
Maybe we can start new threads for the other parts of the patch and
maybe split the patch up a bit. At this point I don't want to commit
major code rearrangements without specific reasons and detailed analysis.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services