Stress test should count failure to repartition as a memory limit exceeded error
The error code for a repartitioning failure recently changed (because it is not strictly a mem limit error). This makes the corresponding change in the stress test. Change-Id: Ie67fabb8d4c0ffc65ac06f35e4a0a5c7a73baddd Reviewed-on: http://gerrit.cloudera.org:8080/3207 Reviewed-by: Tim Armstrong <[email protected]> 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/8f79e9eb Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/8f79e9eb Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/8f79e9eb Branch: refs/heads/master Commit: 8f79e9eb3c0c497f056f22583555b4a2e842eb94 Parents: 5ede8eb Author: Tim Armstrong <[email protected]> Authored: Tue May 24 17:41:39 2016 -0700 Committer: Tim Armstrong <[email protected]> Committed: Tue May 31 23:32:11 2016 -0700 ---------------------------------------------------------------------- tests/stress/concurrent_select.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/8f79e9eb/tests/stress/concurrent_select.py ---------------------------------------------------------------------- diff --git a/tests/stress/concurrent_select.py b/tests/stress/concurrent_select.py index 5047a61..8a82d72 100755 --- a/tests/stress/concurrent_select.py +++ b/tests/stress/concurrent_select.py @@ -796,7 +796,9 @@ class QueryRunner(object): # Exceeding a mem limit may result in the message "cancelled". # https://issues.cloudera.org/browse/IMPALA-2234 - if "memory limit exceeded" in caught_msg or caught_msg == "cancelled": + if "memory limit exceeded" in caught_msg or \ + "repartitioning did not reduce the size of a spilled partition" in caught_msg or \ + caught_msg == "cancelled": report.mem_limit_exceeded = True return
