Dear Dilip,
Thanks for checking my proposal!
> - * Note that if we determine that there's still more data to send, this
> - * function will return control to the caller.
> + * Note that if we determine that there's still more data to send or we are in
> + * the physical replication more, this function will return control to the
> + * caller.
>
> I think in this comment you meant to say
>
> 1. "or we are in physical replication mode and all WALs are not yet replicated"
> 2. Typo /replication more/replication mode
Firstly I considered 2, but I thought 1 seemed to be better.
PSA the updated patch.
Best Regards,
Hayato Kuroda
FUJITSU LIMITED