Hi, I finally had time to properly analyze this, and turns out we've been all just trying to fix symptoms and the actual problems.
All the locking works just fine the way it is in master. The issue with deadlock with apply comes from the wrong handling of the SIGTERM in the apply (we didn't set InterruptPending). I changed the SIGTERM handler in patch 0001 just to die which is actually the correct behavior for apply workers. I also moved the connection cleanup code to the before_shmem_exit callback (similar to walreceiver) and now that part works correctly. The issue with orphaned sync workers is actually two separate issues. First, due to thinko we always searched for sync worker in wait_for_sync_status_change instead of searching for opposite worker as was the intention (i.e. sync worker should search for apply and apply should search for sync). Thats fixed by 0002. And second, we didn't accept any invalidation messages until the whole sync process finished (because it flattens all the remote transactions in the single one) so sync worker didn't learn about subscription changes/drop until it has finished, which I now fixed in 0003. There is still outstanding issue that sync worker will keep running inside the long COPY because the invalidation messages are also not processed until it finishes but all the original issues reported here disappear for me with the attached patches applied. -- Petr Jelinek http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
0003-Receive-invalidation-messages-correctly-in-tablesync.patch
Description: binary/octet-stream
0002-Make-tablesync-worker-exit-when-apply-dies-while-it-.patch
Description: binary/octet-stream
0001-Fix-signal-handling-in-logical-workers.patch
Description: binary/octet-stream
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers