Dear Jeff,

> New v4 series attached.

Thanks for updating the patch. While seeing 0001, I found that GetSubscription()
and SubscriptionConninfo() are called without acquiring the lock for the
subscription. Is there a possibility that DDL commands are executed concurrently
and the cache can be invalidated in-between, thus it causes inconsistent 
results?
I tested concurrent ALTER SUBSCRIPTION SET OWNER command in 
maybe_reread_subscription(),
but the cache is not invalidated thus changing the owner is not reflected.
Is it OK?

Best regards,
Hayato Kuroda
FUJITSU LIMITED

Reply via email to