Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/12086 )
Change subject: Added timeout to run-all-tests ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/12086/1/bin/run-all-tests-timeout-check.sh File bin/run-all-tests-timeout-check.sh: http://gerrit.cloudera.org:8080/#/c/12086/1/bin/run-all-tests-timeout-check.sh@52 PS1, Line 52: "Test run timed out" Explain where to find information? ($IMPALA_TIMEOUT_LOGS_DIR) http://gerrit.cloudera.org:8080/#/c/12086/1/bin/run-all-tests.sh File bin/run-all-tests.sh: http://gerrit.cloudera.org:8080/#/c/12086/1/bin/run-all-tests.sh@34 PS1, Line 34: : ${TIMEOUT_FOR_RUN_ALL_TESTS_MINS:=0} Any reason not to default this to something high, like 20 hours? -- To view, visit http://gerrit.cloudera.org:8080/12086 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iefd3e16a973709f27aeffc7a15bcab8fcdbb349b Gerrit-Change-Number: 12086 Gerrit-PatchSet: 1 Gerrit-Owner: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Comment-Date: Mon, 17 Dec 2018 17:50:21 +0000 Gerrit-HasComments: Yes
