Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10745 )
Change subject: IMPALA-7175: deflake check for failed impalad ...................................................................... Patch Set 3: The thing I'm really after with this patch is to check whether the test winds up killing any impalads. I can see that some tests assume that if they pass, it implies nothing has died. This test tries to check for the case explicitly. One option is to just remove the explicit check and assume that this test or some other test will fail bc of the down impalad. Another option is to figure out why ImpalaCluster thought there were two impalads at the start-- was it some old impalad process that persisted for longer that expected, or something else? -- 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 <vercego...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com> Gerrit-Comment-Date: Thu, 21 Jun 2018 16:00:25 +0000 Gerrit-HasComments: No