Greg Nancarrow <gregn4...@gmail.com> writes:
> I've marked this as "Ready for Committer".
I've pushed this after whacking it around a fair amount. A lot of
that was cosmetic, but one thing that wasn't is that I got rid of the
proposed "which_primary_host" variable. I thought the logic around
that was way too messy and probably buggy. Even if it worked exactly
as intended, I'm dubious that the design intention was good. I think
it makes more sense just to go through the whole server list again
without the restriction to standby servers. In particular, that will
give saner results if the servers' status isn't holding still.
regards, tom lane
