Dear Shubham,

> Thank you for pointing this out and for suggesting the changes. I
> agree with your approach.
> Also, I found a mistake in getopt_long and fixed it in this version of
> the patch.
> The attached patch contains the suggested changes.

Thanks for updating the patch. I think the patch looks mostly OK.

Regarding the test code - I think we should directly refer the pg_subscription 
catalog,
and confirm that subtwophase is 'p'. IIUC, we can wait until
all subscriptions. Subtwophasestate are 'e', by using poll_query_until() and 
[1].

[1]: SELECT count(1) = 0 FROM pg_subscription WHERE subtwophasestate NOT IN 
('e');

Best regards,
Hayato Kuroda
FUJITSU LIMITED

Reply via email to