Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/10745 )
Change subject: IMPALA-7175: deflake check for failed impalad ...................................................................... Patch Set 3: Code-Review+2 I did not notice at first that the issue may no be only +1 daemon running at the start check, but a daemon that became unresponsive but did not terminate at the end check. Now I see that your change reduces both potential sources of flakiness. I have created ticket to make starting/killing Impala processes during tests more robust in general: IMPALA-7197 -- To view, visit http://gerrit.cloudera.org:8080/10745 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I97c6b398e43c6abb1df2b1783c26159137f14fa4 Gerrit-Change-Number: 10745 Gerrit-PatchSet: 3 Gerrit-Owner: Vuk Ercegovac <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]> Gerrit-Comment-Date: Fri, 22 Jun 2018 11:13:35 +0000 Gerrit-HasComments: No
