Re: Corrected documentation of data type for the logical replication message formats.

On Fri, Jul 23, 2021 at 3:23 AM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> I think the patch maybe is not quite correct for all the flags.
>
> For example,
>
> @@ -7607,44 +7615,44 @@ are available since protocol version 3.
> <varlistentry>
> <term>Int8</term>
> <listitem><para>
> - Flags; currently unused (must be 0).
> + Flags (uint8); currently unused.
> </para></listitem>
> </varlistentry>
>
> AFAIK, even though the flags are "unused", the code still insists that
> most (or all? Please check the code) of these flag values MUST be 0,
> so I think that this zero value requirement ought to be indicated in
> the docs using the "Int8(0)" convention [1]. For example,
>
> BEFORE
> Int8
> Flags (uint8); currently unused.
>
> AFTER
> Int8(0)
> Flags (uint8); currently unused.

Thanks for the comments. Attached v6 patch has the changes for the same.

Regards,
Vignesh