-
Notifications
You must be signed in to change notification settings - Fork 666
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
PG16 - Add tests for createdb with ICU_RULES option #7161
Conversation
Codecov Report
@@ Coverage Diff @@
## main #7161 +/- ##
==========================================
- Coverage 93.21% 93.05% -0.16%
==========================================
Files 274 274
Lines 59235 59234 -1
==========================================
- Hits 55213 55120 -93
- Misses 4022 4114 +92 |
01d84c3
to
1054046
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding this test even if we do not propagate (right now). When we propagate CREATE DATABASE
this test should remind the authors to properly deparse these.
CREATE EXTENSION citus; | ||
\c - - - :worker_2_port | ||
CREATE EXTENSION citus; | ||
\c - - - :master_port |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's use citus_add_node
as we might get rid of the master_ ones at some poing
1054046
to
03fd520
Compare
When we create a database, it already needs to be manually created in the workers as well. This new icu_rules option should work as the other options as well. Added a test for that. Relevant PG commit: postgres/postgres@30a53b7
03fd520
to
63338de
Compare
When we create a database, it already needs to be manually created in the workers as well. This new icu_rules option should work as the other options as well. Added a test for that. Relevant PG commit: postgres/postgres@30a53b7
When we create a database, it already needs to be manually created in the workers as well.
This new icu_rules option should work as the other options as well. Added a test for that.
Relevant PG commit:
postgres/postgres@30a53b7
#7138