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