Tim Armstrong has posted comments on this change. Change subject: Stress test should count failure to repartition as a memory limit exceeded error ......................................................................
Patch Set 1: (1 comment) Michael: not quite sure what you mean - there is a non-mem limit error path that I think tries to do what you're suggesting http://gerrit.cloudera.org:8080/#/c/3207/1/tests/stress/concurrent_select.py File tests/stress/concurrent_select.py: Line 800: "repartitioning did not reduce the size of a spilled partition" in caught_msg: > nit: maybe reorder so the memory related ones are first, and then last canc Done -- To view, visit http://gerrit.cloudera.org:8080/3207 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie67fabb8d4c0ffc65ac06f35e4a0a5c7a73baddd Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Casey Ching <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-HasComments: Yes
