Horton

Re: Built-in CTYPE provider

Peter Eisentraut <peter@eisentraut.org>

View thread View on postgr.es
From: Peter Eisentraut <peter@eisentraut.org>
To: Noah Misch <noah@leadboat.com>, pgsql-hackers@postgresql.org, Jeremy Schneider <schneider@ardentperf.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Robert Haas <robertmhaas@gmail.com>, Daniel Verite <daniel@manitou-mail.org>, Jeff Davis <pgsql@j-davis.com>
Date: 2024-07-24T15:27:20Z
Lists: pgsql-hackers 
On 24.07.24 17:19, Noah Misch wrote:
> On Wed, Jul 17, 2024 at 03:03:26PM -0700, Noah Misch wrote:
>> On Wed, Jul 17, 2024 at 08:48:46AM -0700, Jeff Davis wrote:
>>> On Thu, 2024-07-11 at 05:50 -0700, Noah Misch wrote:
>>>>> This is still marked as an open item for 17, but you've already
>>>>> acknowledged[1] that no code changes are necessary in version 17.
>>>>
>>>> Later posts on the thread made that obsolete.  The next step is to
>>>> settle the
>>>> question at https://postgr.es/m/20240706195129.fd@rfd.leadboat.com.
>>>> If that
>>>> conclusion entails a remedy, v17 code changes may be part of that
>>>> remedy.
>>>
>>> This is the first time you've mentioned a code change in version 17. If
>>
>> That's right.
>>
>>> you have something in mind, please propose it. However, this feature
>>> followed the right policies at the time of commit, so there would need
>>> to be a strong consensus to accept such a change.
>>
>> If I'm counting the votes right, you and Tom have voted that the feature's
>> current state is okay, and I and Laurenz have voted that it's not okay.  I
>> still hope more people will vote, to avoid dealing with the tie.  Daniel,
>> Peter, and Jeremy, you're all listed as reviewers on commit f69319f.  Are you
>> willing to vote one way or the other on the question in
>> https://postgr.es/m/20240706195129.fd@rfd.leadboat.com?
> 
> The last vote arrived 6 days ago.  So far, we have votes from Jeff, Noah, Tom,
> Daniel, and Laurenz.  I'll keep the voting open for another 24 hours from now
> or 36 hours after the last vote, whichever comes last.  If that schedule is
> too compressed for anyone, do share.

My opinion is that it is okay to release as is.

Commits

  1. Support PG_UNICODE_FAST locale in the builtin collation provider.

  2. Support Unicode full case mapping and conversion.

  3. Fix test failures when language environment is not UTF-8.

  4. Add unicode_strtitle() for Unicode Default Case Conversion.

  5. Use version for builtin collations.

  6. Fix convert_case(), introduced in 5c40364dd6.

  7. Inline basic UTF-8 functions.

  8. Support C.UTF-8 locale in the new builtin collation provider.

  9. Fix another warning, introduced by 846311051e.

  10. Address more review comments on commit 2d819a08a1.

  11. Fix unreachable code warning from commit 2d819a08a1.

  12. Introduce "builtin" collation provider.

  13. Catalog changes preparing for builtin collation provider.

  14. Unicode case mapping tables and functions.

  15. Add Unicode property tables.

  16. Documentation update for Standard Collations.

  17. Cleanup for unicode-update build target and test.

  18. Shrink Unicode category table.

  19. Make some error strings more generic

  20. pg_upgrade: copy locale and encoding information to new cluster.

  21. Update Unicode data to Unicode 15.0.0

  22. Create a new type category for "internal use" types.