Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15271 )
Change subject: [util] move PeriodWebUIChecker into a separate header file ...................................................................... Patch Set 1: (2 comments) Looks like the TSAN build failed because the Gradle daemon died unexpectedly. Please ask Hao and/or Andrew to debug it further. http://gerrit.cloudera.org:8080/#/c/15271/1/src/kudu/util/webui_checker.h File src/kudu/util/webui_checker.h: PS1: In a follow-up, could you move some of the logic into a new .cc file too? http://gerrit.cloudera.org:8080/#/c/15271/1/src/kudu/util/webui_checker.h@29 PS1, Line 29: #include "kudu/client/client.h" The dependencies on client and mini-cluster modules suggest that util/ isn't a good place for this, or that there needs to be some additional decomposition. -- To view, visit http://gerrit.cloudera.org:8080/15271 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2e6c8e0ebe9e5d03ddf010ea225414af5823becc Gerrit-Change-Number: 15271 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Volodymyr Verovkin <[email protected]> Gerrit-Comment-Date: Sat, 22 Feb 2020 22:22:57 +0000 Gerrit-HasComments: Yes
