From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Call pg_newlocale_from_collation() also with default collation |
Date: | 2022-01-20 09:01:52 |
Message-ID: | E1nATKK-0002dv-L1@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Call pg_newlocale_from_collation() also with default collation
Previously, callers of pg_newlocale_from_collation() did not call it
if the collation was DEFAULT_COLLATION_OID and instead proceeded with
a pg_locale_t of 0. Instead, now we call it anyway and have it return
0 if the default collation was passed. It already did this, so we
just have to adjust the callers. This simplifies all the call sites
and also makes future enhancements easier.
After discussion and testing, the previous comment in pg_locale.c
about avoiding this for performance reasons may have been mistaken
since it was testing a very different patch version way back when.
Reviewed-by: Julien Rouhaud <rjuju123(at)gmail(dot)com>
Discussion: https://www.postgresql.org/message-id/ed3baa81-7fac-7788-cc12-41e3f7917e34@enterprisedb.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/b99ccd2cb2178b62b613edaf427fda8deace5bbf
Modified Files
--------------
src/backend/access/hash/hashfunc.c | 4 +-
src/backend/regex/regc_pg_locale.c | 40 +++++++--------
src/backend/utils/adt/formatting.c | 96 +++++++++++++++++-------------------
src/backend/utils/adt/like.c | 37 +++++++-------
src/backend/utils/adt/like_support.c | 27 +++++-----
src/backend/utils/adt/pg_locale.c | 3 --
src/backend/utils/adt/varchar.c | 26 +++++++---
src/backend/utils/adt/varlena.c | 34 ++++++++-----
8 files changed, 135 insertions(+), 132 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Thomas Munro | 2022-01-20 10:55:17 | pgsql: Try to stabilize reloptions test, again. |
Previous Message | Michael Paquier | 2022-01-20 07:56:48 | pgsql: doc: Mention the level of locks taken on objects in COMMENT |