Thanks for the patches.

1)
I see that there are 2 files now with 053*,  we shall move ours to 054.
ls t/053*
t/053_standby_login_event_trigger.pl
t/053_synchronized_standby_slots_quorum.pl

2)
I feel this test can be removed:
+# PART F: Plain list with first-prefixed slot name still means ALL mode

It is not a test for synchronized_standby_slots, IMO, it is testing
the syncrep parser. Or let me know if my understanding is wrong?

thanks
Shveta


Reply via email to