On Mon, Jul 24, 2017 at 03:40:32PM +0000, Daniel Shahaf wrote: > It seems the real fix would be figuring out a way for 'cleanup' not to > break all locks it sees.
Actually, I missed that cleanup_internal() grabs a WC lock, so the entire point of my argument is moot. The other client manipulating WC state will error out if it runs any DB operation after 'cleanup' has stolen the lock. So I guess I'll just revert my commits :-)