Repository: incubator-impala
Updated Branches:
  refs/heads/master 82290d61a -> 6242478d0


IMPALA-4818: Ensure the same number of tests are run every time

This is a partial revert of Change-Id: I04b4d6db508a26a1a2e4b972b...
from January 2014. That commit caused test_cancel_insert to run
different random tests each time it was run; the number of tests run
could vary between 1 and 9. This commit fixes the number at 9, and the
same 9 every time, to avoid test flakiness.

Change-Id: I22cecfbe7c9a102f788d01eb80aa188579ef6d7e
Reviewed-on: http://gerrit.cloudera.org:8080/5784
Reviewed-by: Michael Brown <[email protected]>
Reviewed-by: Alex Behm <[email protected]>
Tested-by: Impala Public Jenkins


Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/87cec889
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/87cec889
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/87cec889

Branch: refs/heads/master
Commit: 87cec889d091448d8cc0e0b00070fe324e97551f
Parents: 82290d6
Author: Jim Apple <[email protected]>
Authored: Tue Jan 24 15:13:56 2017 -0800
Committer: Impala Public Jenkins <[email protected]>
Committed: Thu Feb 9 19:30:22 2017 +0000

----------------------------------------------------------------------
 tests/query_test/test_cancellation.py | 3 ---
 1 file changed, 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/87cec889/tests/query_test/test_cancellation.py
----------------------------------------------------------------------
diff --git a/tests/query_test/test_cancellation.py 
b/tests/query_test/test_cancellation.py
index edb095f..bb1fc0d 100644
--- a/tests/query_test/test_cancellation.py
+++ b/tests/query_test/test_cancellation.py
@@ -20,7 +20,6 @@
 
 import pytest
 import threading
-from random import choice
 from time import sleep
 from tests.beeswax.impala_beeswax import ImpalaBeeswaxException
 from tests.common.test_vector import ImpalaTestDimension
@@ -195,8 +194,6 @@ class TestCancellationSerial(TestCancellation):
     # Don't run across all cancel delay options unless running in exhaustive 
mode
     if cls.exploration_strategy() != 'exhaustive':
       cls.ImpalaTestMatrix.add_constraint(lambda v: 
v.get_value('cancel_delay') in [3])
-      cls.ImpalaTestMatrix.add_constraint(lambda v: v.get_value('query') ==\
-          choice(QUERIES.keys()))
 
   @pytest.mark.execute_serially
   def test_cancel_insert(self, vector):

Reply via email to