Don't allow PG11 travis failures anymore

We made PG11 builds optional when we had an issue
with mx isolation test that we could not solve back then.

This commit solves the issue with a workaround by running
start_metadata_sync_to_node outside the transaction block.
pull/2491/head
Murat Tuncer 2018-10-19 11:41:13 +03:00
parent 87817aec9d
commit b08106b5cf
3 changed files with 21 additions and 7 deletions

View File

@ -21,8 +21,6 @@ matrix:
- env: PGVERSION=9.6 - env: PGVERSION=9.6
- env: PGVERSION=10 - env: PGVERSION=10
- env: PGVERSION=11 - env: PGVERSION=11
allow_failures:
- env: PGVERSION=11
before_install: before_install:
- git clone -b v0.7.9 --depth 1 https://github.com/citusdata/tools.git - git clone -b v0.7.9 --depth 1 https://github.com/citusdata/tools.git
- sudo make -C tools install - sudo make -C tools install

View File

@ -19,9 +19,17 @@ setup
SELECT citus.replace_isolation_tester_func(); SELECT citus.replace_isolation_tester_func();
SELECT citus.refresh_isolation_tester_prepared_statement(); SELECT citus.refresh_isolation_tester_prepared_statement();
SELECT start_metadata_sync_to_node('localhost', 57637); -- start_metadata_sync_to_node can not be run inside a transaction block
SELECT start_metadata_sync_to_node('localhost', 57638); -- following is a workaround to overcome that
-- port numbers are hard coded at the moment
SELECT master_run_on_worker(
ARRAY['localhost']::text[],
ARRAY[57636]::int[],
ARRAY[format('SELECT start_metadata_sync_to_node(''%s'', %s)', nodename, nodeport)]::text[],
false)
FROM pg_dist_node;
SET citus.shard_replication_factor TO 1; SET citus.shard_replication_factor TO 1;
SET citus.replication_model to streaming; SET citus.replication_model to streaming;

View File

@ -20,8 +20,16 @@ setup
SELECT citus.replace_isolation_tester_func(); SELECT citus.replace_isolation_tester_func();
SELECT citus.refresh_isolation_tester_prepared_statement(); SELECT citus.refresh_isolation_tester_prepared_statement();
SELECT start_metadata_sync_to_node('localhost', 57637); -- start_metadata_sync_to_node can not be run inside a transaction block
SELECT start_metadata_sync_to_node('localhost', 57638); -- following is a workaround to overcome that
-- port numbers are hard coded at the moment
SELECT master_run_on_worker(
ARRAY['localhost']::text[],
ARRAY[57636]::int[],
ARRAY[format('SELECT start_metadata_sync_to_node(''%s'', %s)', nodename, nodeport)]::text[],
false)
FROM pg_dist_node;
SET citus.replication_model to streaming; SET citus.replication_model to streaming;
CREATE TABLE ref_table(user_id int, value_1 int); CREATE TABLE ref_table(user_id int, value_1 int);