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 <[email protected]> Reviewed-by: Dan Hecht <[email protected]> Tested-by: Tim Armstrong <[email protected]> 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 <[email protected]> Authored: Mon Aug 28 16:56:41 2017 -0700 Committer: Tim Armstrong <[email protected]> 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
