Merge pull request #1426 from citusdata/better_comment_for_tests

Add comment to subquery regression test file
pull/1429/head
Önder Kalacı 2017-05-22 11:12:20 +03:00 committed by GitHub
commit 1ea96b626b
2 changed files with 10 additions and 6 deletions

View File

@ -3,9 +3,11 @@
-- regression tests to cover more cases -- regression tests to cover more cases
-- the tables that are used depends to multi_insert_select_behavioral_analytics_create_table.sql -- the tables that are used depends to multi_insert_select_behavioral_analytics_create_table.sql
-- --
-- We don't need shard id sequence here, so commented out to prevent conflicts with concurrent tests --- We don't need shard id sequence here given that we're not creating any shards, so not writing it at all
-- ALTER SEQUENCE pg_catalog.pg_dist_shardid_seq RESTART 1430000; -- The following line is intended to force Citus to NOT use router planner for the tests in this
-- ALTER SEQUENCE pg_catalog.pg_dist_jobid_seq RESTART 1430000; -- file. The motivation for doing this is to make sure that single-task queries can be planned
-- by non-router code-paths. Thus, this flag should NOT be used in production. Otherwise, the actual
-- router queries would fail.
SET citus.enable_router_execution TO FALSE; SET citus.enable_router_execution TO FALSE;
------------------------------------ ------------------------------------
-- Vanilla funnel query -- Vanilla funnel query

View File

@ -4,10 +4,12 @@
-- the tables that are used depends to multi_insert_select_behavioral_analytics_create_table.sql -- the tables that are used depends to multi_insert_select_behavioral_analytics_create_table.sql
-- --
-- We don't need shard id sequence here, so commented out to prevent conflicts with concurrent tests --- We don't need shard id sequence here given that we're not creating any shards, so not writing it at all
-- ALTER SEQUENCE pg_catalog.pg_dist_shardid_seq RESTART 1430000;
-- ALTER SEQUENCE pg_catalog.pg_dist_jobid_seq RESTART 1430000;
-- The following line is intended to force Citus to NOT use router planner for the tests in this
-- file. The motivation for doing this is to make sure that single-task queries can be planned
-- by non-router code-paths. Thus, this flag should NOT be used in production. Otherwise, the actual
-- router queries would fail.
SET citus.enable_router_execution TO FALSE; SET citus.enable_router_execution TO FALSE;
------------------------------------ ------------------------------------