Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12230 )
Change subject: IMPALA-8089: Fixing run-all-tests timeout raciness. ...................................................................... IMPALA-8089: Fixing run-all-tests timeout raciness. We observed some failures in "pkill -P $TIMEOUT_PID". pkill will fail if there are no child processes, which could happen if there is, at that point in time, no "sleep" process running. To fix this, we simply ignore the error condition here. I added some debugging to run-all-tests-timeout-check.sh as well. I'm not 100% certain this is the right fix. Change-Id: Ieccfda933f526c116e49c46bf34f7245b357cb07 Reviewed-on: http://gerrit.cloudera.org:8080/12230 Reviewed-by: Impala Public Jenkins <[email protected]> Reviewed-by: Bikramjeet Vig <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M bin/run-all-tests-timeout-check.sh M bin/run-all-tests.sh 2 files changed, 7 insertions(+), 3 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified Bikramjeet Vig: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12230 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ieccfda933f526c116e49c46bf34f7245b357cb07 Gerrit-Change-Number: 12230 Gerrit-PatchSet: 3 Gerrit-Owner: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]>
