On Sat, Dec 08, 2018 at 06:24:35PM -0800, Noah Misch wrote:
> pg_recvlogical_upto() has the ability to make timeout non-fatal, by calling it
> in an array context. No in-tree test uses that. Out-of-tree code using that
> feature would likely benefit from the ability to set timeout duration. Hence,
> I'm inclined not remove the timeout duration parameter.
No objections with this line of thoughts. The rest of the patch looks
fine to me.
--
Michael