pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
pgsql: Special case C_COLLATION_OID in pg_newlocale_from_collation().
2+ messages / 1 participants
[nested] [flat]

* pgsql: Special case C_COLLATION_OID in pg_newlocale_from_collation().
@ 2025-11-05 00:49  Jeff Davis <jdavis@postgresql.org>
  0 siblings, 0 replies; 2+ messages in thread

From: Jeff Davis @ 2025-11-05 00:49 UTC (permalink / raw)
  To: pgsql-committers@lists.postgresql.org

Special case C_COLLATION_OID in pg_newlocale_from_collation().

Allow pg_newlocale_from_collation(C_COLLATION_OID) to work even if
there's no catalog access, which some extensions expect.

Not known to be a bug without extensions involved, but backport to 18.

Also corrects an issue in master with dummy_c_locale (introduced in
commit 5a38104b36) where deterministic was not set. That wasn't a bug,
but could have been if that structure was used more widely.

Reported-by: Alexander Kukushkin <cyberdemn@gmail.com>
Reviewed-by: Alexander Kukushkin <cyberdemn@gmail.com>
Discussion: https://postgr.es/m/CAFh8B=nj966ECv5vi_u3RYij12v0j-7NPZCXLYzNwOQp9AcPWQ@mail.gmail.com
Backpatch-through: 18

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/d115de9d89164e87269d73d4f0f1368f06ebdd5e

Modified Files
--------------
src/backend/regex/regc_pg_locale.c | 36 +++++-------------------------------
src/backend/utils/adt/pg_locale.c  | 13 +++++++++++++
2 files changed, 18 insertions(+), 31 deletions(-)



^ permalink  raw  reply  [nested|flat] 2+ messages in thread

* pgsql: Special case C_COLLATION_OID in pg_newlocale_from_collation().
@ 2025-11-05 00:49  Jeff Davis <jdavis@postgresql.org>
  0 siblings, 0 replies; 2+ messages in thread

From: Jeff Davis @ 2025-11-05 00:49 UTC (permalink / raw)
  To: pgsql-committers@lists.postgresql.org

Special case C_COLLATION_OID in pg_newlocale_from_collation().

Allow pg_newlocale_from_collation(C_COLLATION_OID) to work even if
there's no catalog access, which some extensions expect.

Not known to be a bug without extensions involved, but backport to 18.

Also corrects an issue in master with dummy_c_locale (introduced in
commit 5a38104b36) where deterministic was not set. That wasn't a bug,
but could have been if that structure was used more widely.

Reported-by: Alexander Kukushkin <cyberdemn@gmail.com>
Reviewed-by: Alexander Kukushkin <cyberdemn@gmail.com>
Discussion: https://postgr.es/m/CAFh8B=nj966ECv5vi_u3RYij12v0j-7NPZCXLYzNwOQp9AcPWQ@mail.gmail.com
Backpatch-through: 18

Branch
------
REL_18_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/3ebea75f9afab4f526934100b4ba747e9d95bba8

Modified Files
--------------
src/backend/utils/adt/pg_locale.c | 14 ++++++++++++++
1 file changed, 14 insertions(+)



^ permalink  raw  reply  [nested|flat] 2+ messages in thread


end of thread, other threads:[~2025-11-05 00:49 UTC | newest]

Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2025-11-05 00:49 pgsql: Special case C_COLLATION_OID in pg_newlocale_from_collation(). Jeff Davis <jdavis@postgresql.org>
2025-11-05 00:49 pgsql: Special case C_COLLATION_OID in pg_newlocale_from_collation(). Jeff Davis <jdavis@postgresql.org>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox