Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15893 )
Change subject: KUDU-3112. Fix WaitForBind method for checking service status ...................................................................... Patch Set 4: > Build Successful > > http://jenkins.kudu.apache.org/job/kudu-gerrit/21643/ : SUCCESS Thank you for addressing these comments. BTW, would the tests pass on aarch64 if you apply the following patch: https://gerrit.cloudera.org/#/c/15917/ ? If that works, we could keep the original semantics of WaitForBind() and this patch wouldn't be needed. -- To view, visit http://gerrit.cloudera.org:8080/15893 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib79297e0eb59cc96a91c6e301f6a70ba123f4644 Gerrit-Change-Number: 15893 Gerrit-PatchSet: 4 Gerrit-Owner: liusheng <liusheng2...@gmail.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: liusheng <liusheng2...@gmail.com> Gerrit-Comment-Date: Thu, 14 May 2020 20:53:14 +0000 Gerrit-HasComments: No