[PATCH v9 1/5] Fix error inconsistency in older ICU versions.

ok


[PATCH v9 2/5] initdb: replace check_icu_locale() with
 default_icu_locale().

I would keep the #ifdef USE_ICU inside the lower-level function default_icu_locale(), like it was before, so that the higher-level setlocales() doesn't need to know about it.

Otherwise ok.


[PATCH v9 3/5] initdb: emit message when using default ICU locale.

ok


[PATCH v9 4/5] Validate ICU locales.

Also here, let's keep the #ifdef USE_ICU in the lower-level function and move more logic in there. Otherwise you have to repeat various things in DefineCollation() and createdb().

I'm not sure we need the IsBinaryUpgrade checks. Users can set icu_validation_level on the target instance if they don't want that.



Reply via email to