>> I agree with first version, and attached the updatedpatchwhich are >> modified so that it supports simple multiple sync replication you >>suggested. >> (but test cases are not included yet.)
I have tried for some basic in-built test cases for multisync rep.
Please have look and correct me if i am wrong and suggest remaining test cases.
So the interesting part of this patch is 006_sync_rep.pl. I think that you had better build something on top of my patch as a separate patch. This would make things clearer.
+my $result = $node_master->psql('postgres', "select application_name, sync_state from pg_stat_replication;"); +print "$result \n"; +is($result, "standby_1|sync\nstandby_2|sync\nstandby_3|potential", 'checked for sync standbys state initially');
Now regarding the test, you visibly got the idea, though I think that we'd want to update a bit the parameters of postgresql.conf and re-run those queries a couple of times, that's cheaper than having to re-create new cluster nodes all the time, so just create a base, then switch s_s_names a bit, and query pg_stat_replication, and you are already doing the latter.
Also, please attach patches directly to your emails. When loading something on nabble this is located only there and not within postgresql.org which would be annoying if nabble disappears at some point. You would also want to use directly an email client and interact with the community mailing lists this way instead of going through the nabble's forum-like interface (never used it, not really willing to use it, but I guess that it is similar to that).
I am attaching what you posted on this email for the archive's sake.