Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12103 )
Change subject: IMPALA-7990: make test_lifecycle_failures more permissive ...................................................................... IMPALA-7990: make test_lifecycle_failures more permissive test_lifecycle_failures runs a query with a debug action set to cause fragment thread creation to fail 50% of the time and checks that the query either succeeded or failed with an expected error. Generally, the failure happens early enough that the query hasn't yet reached FINISHED, but its also correct under Impala's query lifecycle model for the query to reach FINISHED before the error is hit, in which case it will be returned by a call to fetch(). Previously, the expected error in the test only handled the first case. This patch modifies it to be more permissive in order to handle both cases. Testing: - Ran test_lifecycle_failures in a loop 200 times without a failure. Previously it repro-ed for me ~10% of the time. Change-Id: I2da4ca95692b445e05796105d3b7d14e51fece88 Reviewed-on: http://gerrit.cloudera.org:8080/12103 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M tests/failure/test_failpoints.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/12103 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2da4ca95692b445e05796105d3b7d14e51fece88 Gerrit-Change-Number: 12103 Gerrit-PatchSet: 2 Gerrit-Owner: Thomas Marshall <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
