Re: Rotten parts of src/backend/replication/README

On Tue, Jun 30, 2020 at 10:11:04AM +0200, Peter Eisentraut wrote:
> On 2020-05-04 07:50, Michael Paquier wrote:
>> Yeah. The years have visibly proved that the README had updates when
>> it came to the general descriptions of the libpqwalreceiver interface,
>> but that we had better consolidate the header to give some
>> documentation to whoever plays with this interface. Attached is a
>> patch to address all that, where I simplified the README and added
>> some description to all the callbacks. Thoughts are welcome. I'll
>> add that to the next CF. Now I don't see any actual problems in
>> getting that on HEAD before v13 forks. But let's gather more opinions
>> first.
>
> This patch looks good to me.

Thanks for the review, Peter. After an extra read, the description
of walrcv_create_slot_fn was not complete (missed the end of the
sentence to say that NULL is returned for a physical slot) and had a
grammar mistake. So I have fixed this part, and applied the patch on
HEAD. Perhaps things could be improved further more, so if anybody
has any suggestion please feel free.
--
Michael