Re: Corrected documentation of data type for the logical replication message formats. - Mailing list pgsql-hackers

From vignesh C
Subject Re: Corrected documentation of data type for the logical replication message formats.
Date
Msg-id CALDaNm0mfUT8A_Tu0hMEjfkvvzqJF_uENcc7Nno9en9E_aP7Zw@mail.gmail.com
Whole thread Raw
In response to Re: Corrected documentation of data type for the logical replication message formats.  ("Euler Taveira" <euler@eulerto.com>)
List pgsql-hackers


On Tue, May 11, 2021 at 9:09 AM Euler Taveira <euler@eulerto.com> wrote:
>
> On Mon, May 10, 2021, at 10:45 AM, vignesh C wrote:
>
> I agree to specifying the actual dataypes like XLogRecPtr for lsn,
> TimestampTz for timestamp, TransactionId for xid and Oid for the
> object id. Attached v2 patch which is changed on similar lines.
> Thoughts?
>
> Perhaps I didn't make myself clear, I didn't suggest to replace the actual
> message data types [1] with the internal representation. We could probably
> expand the description to include the internal representation. Hence, it is
> less confusing that the actual text. Peter suggested the same in a previous
> email.
>
> Although, "Message Data Types" is one section before "Message Formats", it is
> probably intuitive that the data type for each message refer to the previous
> section. However, it is not so clear three section later. A sentence like
>
> The base data types used are described in the section "Messages Data Types".
>
> at the first paragraph could help understand what these data types refer to
> (and also add a link to the data types section).

I have included this at the beginning, the same is available in the patch posted at [1].
[1] - https://www.postgresql.org/message-id/CALDaNm2QrB-_96ohonQs-YADC9Puk3caXjn%2B2UYZwxAkX%3DREQQ%40mail.gmail.com

Regards,
Vignesh

pgsql-hackers by date:

Previous
From: vignesh C
Date:
Subject: Re: Corrected documentation of data type for the logical replication message formats.
Next
From: Bruce Momjian
Date:
Subject: Re: PG 14 release notes, first draft