Re: [patch] ENUM errdetail should mention bytes, not chars

2020年10月27日(火) 20:00 Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>:
>
> On 2020-10-19 06:34, Julien Rouhaud wrote:
> >> ERROR: invalid enum label "ああああああああああああああああああああああ"
> >> DETAIL: Labels must be 63 characters or less.
> >>
> >> Attached trivial patch changes the message to:
> >>
> >> DETAIL: Labels must be 63 bytes or less.
> >>
> >> This matches the documentation, which states:
> >>
> >> The length of an enum value's textual label is limited by the NAMEDATALEN
> >> setting compiled into PostgreSQL; in standard builds this means at most
> >> 63 bytes.
> >>
> >> https://www.postgresql.org/docs/current/datatype-enum.html
> >>
> >> I don't see any particular need to backpatch this.
> >
> > Indeed the message is wrong, and patch LGTM.
>
> Committed.

Thanks!

> Btw., the patch didn't update the regression test output.

Whoops... /me hangs head in shame and slinks away...

Regards

Ian Barwick

--
EnterpriseDB: https://www.enterprisedb.com