From a2fb5a84f108aacffafd3a2d63e797d098f1f7dc Mon Sep 17 00:00:00 2001 From: Jason Petersen Date: Thu, 16 Aug 2018 19:37:14 -0600 Subject: [PATCH 1/3] Disable JIT during PostgreSQL 11 test runs It's causing problems with one of our tests. --- src/test/regress/pg_regress_multi.pl | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/test/regress/pg_regress_multi.pl b/src/test/regress/pg_regress_multi.pl index 05771a193..9ba5aff72 100755 --- a/src/test/regress/pg_regress_multi.pl +++ b/src/test/regress/pg_regress_multi.pl @@ -313,6 +313,11 @@ push(@pgOptions, '-c', "citus.remote_task_check_interval=1ms"); push(@pgOptions, '-c', "citus.shard_replication_factor=2"); push(@pgOptions, '-c', "citus.node_connection_timeout=${connectionTimeout}"); +if ($majorversion >= 11) +{ + push(@pgOptions, '-c', "jit=off"); +} + if ($useMitmproxy) { # make tests reproducible by never trying to negotiate ssl From c4e2349b80edc6d6554c03f7816dd795322dfd13 Mon Sep 17 00:00:00 2001 From: Jason Petersen Date: Thu, 16 Aug 2018 19:37:37 -0600 Subject: [PATCH 2/3] Mark failing PostgreSQL 11 test as ignored This commit should be reverted once a new PostgreSQL 11 beta is available: it's due to a bug in the partitioning code which has been fixed in REL_11_STABLE but (not yet) a released tag. --- src/test/regress/multi_schedule | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/test/regress/multi_schedule b/src/test/regress/multi_schedule index 31949fe9f..4b0bdf403 100644 --- a/src/test/regress/multi_schedule +++ b/src/test/regress/multi_schedule @@ -37,6 +37,8 @@ test: multi_insert_select_window multi_shard_update_delete window_functions dml_ # ---------- # Tests for partitioning support # ---------- +# TODO: remove the following ignore when PostgreSQL 11 has a new beta +ignore: multi_partitioning test: multi_partitioning_utils multi_partitioning From fd32c3590bfaf37a217ac041e1e0be08a7cded0c Mon Sep 17 00:00:00 2001 From: Jason Petersen Date: Thu, 16 Aug 2018 19:38:55 -0600 Subject: [PATCH 3/3] Mark PostgreSQL 11 builds as required We will no longer accept regressions in our support for PostgreSQL 11. --- .travis.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index 120b33f1f..3c5517fdf 100644 --- a/.travis.yml +++ b/.travis.yml @@ -21,8 +21,6 @@ matrix: - env: PGVERSION=9.6 - env: PGVERSION=10 - env: PGVERSION=11 - allow_failures: - - env: PGVERSION=11 before_install: - git clone -b v0.7.9 --depth 1 https://github.com/citusdata/tools.git - sudo make -C tools install