You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by ta...@apache.org on 2016/08/19 01:22:28 UTC

incubator-impala git commit: Fix stress test runner bug introduced by IMPALA-3969

Repository: incubator-impala
Updated Branches:
  refs/heads/master 4d03ed40c -> 084982714


Fix stress test runner bug introduced by IMPALA-3969

In some cases if common_query_options was not specified, the stress test
runner could fail during the mem_limit binary search phase.

Change-Id: I86484bb7c92ae1069f6a07cf3ea5027740364150
Reviewed-on: http://gerrit.cloudera.org:8080/4019
Reviewed-by: Michael Brown <mi...@cloudera.com>
Reviewed-by: Dan Hecht <dh...@cloudera.com>
Tested-by: Internal 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/08498271
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/08498271
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/08498271

Branch: refs/heads/master
Commit: 0849827147b1b4d415b32c3a569946820347aa98
Parents: 4d03ed4
Author: Tim Armstrong <ta...@cloudera.com>
Authored: Wed Aug 17 12:34:49 2016 -0700
Committer: Internal Jenkins <cl...@gerrit.cloudera.org>
Committed: Fri Aug 19 01:20:01 2016 +0000

----------------------------------------------------------------------
 tests/stress/concurrent_select.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/08498271/tests/stress/concurrent_select.py
----------------------------------------------------------------------
diff --git a/tests/stress/concurrent_select.py b/tests/stress/concurrent_select.py
index dc06503..a43b402 100755
--- a/tests/stress/concurrent_select.py
+++ b/tests/stress/concurrent_select.py
@@ -677,7 +677,6 @@ class Query(object):
     self.required_mem_mb_without_spilling = None
     self.solo_runtime_secs_with_spilling = None
     self.solo_runtime_secs_without_spilling = None
-    self.common_query_options = {}
 
   def __repr__(self):
     return dedent("""
@@ -702,6 +701,7 @@ class QueryRunner(object):
     self.use_kerberos = False
     self.result_hash_log_dir = gettempdir()
     self.check_if_mem_was_spilled = False
+    self.common_query_options = {}
 
   def connect(self):
     self.impalad_conn = self.impalad.impala.connect(impalad=self.impalad)