Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14741 )
Change subject: IMPALA-9165: Add timeout for create-load-data.sh ...................................................................... IMPALA-9165: Add timeout for create-load-data.sh This converts the existing bin/run-all-tests-timeout-check.sh to a more generic bin/script-timeout-check.sh. It uses this new script for both bin/run-all-tests.sh and testdata/bin/create-load-data.sh. The new script takes two arguments: -timeout : timeout in minutes -script_name : name of the calling script The script_name is used in debugging output / output filenames to make it clear what timed out. The run-all-tests.sh timeout remains the same. testdata/bin/create-load-data.sh uses a 2.5 hour timeout. This should help debug the issue in IMPALA-9165, because at least the logs would be preserved on the Jenkins job. Testing: - Tested the timeout script by hand with a caller script that sleeps longer than the timeout - Ran a gerrit-verify-dryrun-external Change-Id: I19d76bd8850c7d4b5affff4d21f32d8715a382c6 Reviewed-on: http://gerrit.cloudera.org:8080/14741 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Joe McDonnell <[email protected]> --- D bin/run-all-tests-timeout-check.sh M bin/run-all-tests.sh A bin/script-timeout-check.sh M testdata/bin/create-load-data.sh 4 files changed, 129 insertions(+), 71 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/14741 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I19d76bd8850c7d4b5affff4d21f32d8715a382c6 Gerrit-Change-Number: 14741 Gerrit-PatchSet: 4 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Sahil Takiar <[email protected]>
