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 2017/08/29 20:01:27 UTC
incubator-impala git commit: IMPALA-2234: remove workaround from
stress test
Repository: incubator-impala
Updated Branches:
refs/heads/master 8149d0e57 -> 6a9ca345b
IMPALA-2234: remove workaround from stress test
We believe that IMPALA-2234 has probably been fixed at some point -
either by IMPALA-3200 or other query lifecycle changes. Let's remove the
workaround from the stress test script to confirm that it's no longer
needed.
Change-Id: Ic154ce245195e8ef9af136da7ab58e82b5b4354e
Reviewed-on: http://gerrit.cloudera.org:8080/7874
Reviewed-by: Lars Volker <lv...@cloudera.com>
Reviewed-by: Dan Hecht <dh...@cloudera.com>
Tested-by: Tim Armstrong <ta...@cloudera.com>
Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/6a9ca345
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/6a9ca345
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/6a9ca345
Branch: refs/heads/master
Commit: 6a9ca345b0dc913f82c0a6ea4d793153732e3916
Parents: 8149d0e
Author: Tim Armstrong <ta...@cloudera.com>
Authored: Mon Aug 28 16:56:41 2017 -0700
Committer: Tim Armstrong <ta...@cloudera.com>
Committed: Tue Aug 29 20:01:17 2017 +0000
----------------------------------------------------------------------
tests/stress/concurrent_select.py | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/6a9ca345/tests/stress/concurrent_select.py
----------------------------------------------------------------------
diff --git a/tests/stress/concurrent_select.py b/tests/stress/concurrent_select.py
index d250fe1..bc1a530 100755
--- a/tests/stress/concurrent_select.py
+++ b/tests/stress/concurrent_select.py
@@ -941,13 +941,11 @@ class QueryRunner(object):
fetch_and_set_profile(cursor, report)
caught_msg = str(caught_exception).lower().strip()
- # Exceeding a mem limit may result in the message "cancelled". See IMPALA-2234
if "memory limit exceeded" in caught_msg or \
"repartitioning did not reduce the size of a spilled partition" in caught_msg or \
"failed to get minimum memory reservation" in caught_msg or \
"minimum memory reservation is greater than" in caught_msg or \
- "minimum memory reservation needed is greater than" in caught_msg or \
- caught_msg == "cancelled":
+ "minimum memory reservation needed is greater than" in caught_msg:
report.mem_limit_exceeded = True
return