Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PG17 compatibility: add COLLPROVIDER_BUILTIN option and fix tests #7752

Merged
merged 2 commits into from
Nov 19, 2024

Conversation

naisila
Copy link
Member

@naisila naisila commented Nov 18, 2024

In PG17 adds builtin C.UTF-8 locale option, we add it in the code to avoid "unknown collation provider" in vanilla tests.

Relevant PG commit:
postgres/postgres@f69319f
f69319f2f1fb16eda4b535bcccec90dff3a6795e

Also in PG17, colliculocale, daticulocale renamed to colllocale, datlocale
Here we fix the following tests to avoid alternative output
pg15 pg16 multi_mx_create_table multi_schema_support

Relevant PG commit:
postgres/postgres@f696c0c
f696c0cd5f299f1b51e214efc55a22a782cc175d

Note to reviewer: First three commits are for PG17 testing purposes. Only the last 2 commits will be merged. Also, I wasn't sure whether I should add tests here.

Copy link

codecov bot commented Nov 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (release-13.0@a93887b). Learn more about missing BASE report.

Additional details and impacted files
@@               Coverage Diff               @@
##             release-13.0    #7752   +/-   ##
===============================================
  Coverage                ?   89.64%           
===============================================
  Files                   ?      274           
  Lines                   ?    59583           
  Branches                ?     7436           
===============================================
  Hits                    ?    53415           
  Misses                  ?     4036           
  Partials                ?     2132           
---- 🚨 Try these New Features:

@naisila naisila changed the title PG17 compatibility: fix tests related to builtin collation provider PG17 compatibility: add builtin collation provider option and fix tests Nov 18, 2024
@naisila naisila changed the title PG17 compatibility: add builtin collation provider option and fix tests PG17 compatibility: add COLLPROVIDER_BUILTIN option and fix tests Nov 18, 2024
…tests

Fix pg15 pg16 multi_mx_create_table multi_schema_support

Relevant PG commit:
postgres/postgres@f696c0c
f696c0cd5f299f1b51e214efc55a22a782cc175d

(cherry picked from commit 17a2ed0)
Relevant PG commit:
f69319f2f1fb16eda4b535bcccec90dff3a6795e
postgres/postgres@f69319f

(cherry picked from commit 6c12b10)
@naisila naisila merged commit ed13700 into release-13.0 Nov 19, 2024
120 checks passed
@naisila naisila deleted the naisila/pg17_colliculocale branch November 19, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants