On 09.03.23 21:17, Jeff Davis wrote:
>> Personally, I'm not on board with this behavior:
>>
>> => CREATE COLLATION test (provider = icu, locale =
>> 'de@collation=phonebook');
>> NOTICE: 00000: using language tag "de-u-co-phonebk" for locale
>> "de@collation=phonebook"
>>
>> I mean, maybe that is a thing we want to do somehow sometime, to
>> migrate
>> people to the "new" spellings, but the old ones aren't wrong.
>
> I see what you mean; I'm not sure the best thing to do here. We are
> adjusting the string passed by the user, and it feels like some users
> might want to know that. It's a NOTICE, not a WARNING, so it's not
> meant to imply that it's wrong.
For clarification, I wasn't complaining about the notice, but about the
automatic conversion from old-style ICU locale ID to language tag.
>> It also doesn't appear to address
>> how to handle ICU before version 54.
>
> Do you have a specific concern here?
What we had discussed a while ago in one of these threads is that ICU
before version 54 do not support keyword lists, and we have custom code
to do that parsing ourselves, but we don't have code to do the same for
language tags. Therefore, if I understand this right, if we
automatically convert ICU locale IDs to language tags, as shown above,
then we break support for such locales in those older ICU versions.