From: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
---|---|
To: | Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: doc: Move enum storage commentary to top of section |
Date: | 2022-07-06 17:34:58 |
Message-ID: | CAKFQuwaCWPj7AGuARef65vNuY1gGOE-rY=Pijtka4Uj=yJjRZA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Jul 6, 2022 at 10:24 AM Matthias van de Meent <
boekewurm+postgres(at)gmail(dot)com> wrote:
> On Thu, 9 Jun 2022 at 18:12, David G. Johnston
> <david(dot)g(dot)johnston(at)gmail(dot)com> wrote:
> >
> > Per suggestion over on -docs:
> >
> >
> https://www.postgresql.org/message-id/BL0PR06MB4978F6C0B69F3F03AEBED0FBB3C29@BL0PR06MB4978.namprd06.prod.outlook.com
>
> I agree with moving the size of an enum into the top, but I don't
> think that the label length documentation also should be included in
> the top section. It seems excessively detailed for that section with
> its reference to NAMEDATALEN.
>
> -Matthias
>
Agreed.
Tangentially: It does seem a bit unusual to call the fact that the values
both case-sensitive and limited to the length of a system identifier an
implementation detail. But if anything the length is more of one than the
case-sensitivity. Specifying NAMEDATALEN here seems like it breaks
encapsulation, it could refer by comparison to an identifier and only those
that care can learn how that length might be changed in a custom build of
PostgreSQL.
David J.
From | Date | Subject | |
---|---|---|---|
Next Message | Alexander Korotkov | 2022-07-06 17:45:44 | Re: Custom tuplesorts for extensions |
Previous Message | Matthias van de Meent | 2022-07-06 17:23:52 | Re: doc: Move enum storage commentary to top of section |