Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> I think the attached might be a little tidier. Thoughts?
I'm not really thrilled at the idea of calling
IsPostmasterChildWalSender for every child whether or not it will have
any impact on the decision. That involves touching shared memory which
can be rather expensive (see previous discussions about shared cache
lines and so forth).
The existing coding is pretty ugly, I agree.
regards, tom lane