Re: pgsql: Use ICU by default at initdb time.

From: Jeff Davis <pgsql(at)j-davis(dot)com>
To: Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgsql: Use ICU by default at initdb time.
Date: 2023-03-10 18:58:49
Message-ID: 437cb29641d176ccfb9d631351582fdd96842d8d.camel@j-davis.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Fri, 2023-03-10 at 07:48 -0800, Jeff Davis wrote:
> On Fri, 2023-03-10 at 15:48 +0100, Peter Eisentraut wrote:
> > Yes, of course.  So we can't really do what I was thinking of.
>
> OK, I plan to commit something like the patch in this thread soon. I
> just need to add an explanatory comment.

Committed a slightly narrower fix that derives the default encoding the
same way for both libc and ICU; except that ICU still uses UTF-8 for
C/POSIX/--no-locale (because ICU doesn't work with SQL_ASCII).

That seemed more consistent with the comments around
pg_get_encoding_from_locale() and it was also easier to document the -E
switch in initdb.

I'll keep an eye on the buildfarm to see if this fixes the problem or
causes other issues. But it seems like the right change.

Regards,
Jeff Davis

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Jeff Davis 2023-03-10 19:44:08 Re: pgsql: Add standard collation UNICODE
Previous Message Tom Lane 2023-03-10 18:52:57 pgsql: Ensure COPY TO on an RLS-enabled table copies no more than it sh

Browse pgsql-hackers by date

  From Date Subject
Next Message Melanie Plageman 2023-03-10 19:51:13 Re: pg_stat_bgwriter.buffers_backend is pretty meaningless (and more?)
Previous Message Tom Lane 2023-03-10 18:57:55 Re: RLS makes COPY TO process child tables