Alexey Serbin has posted comments on this change. Change subject: [web-ui] KUDU-1619 Separate tables for live and dead tservers on /tablet-servers ......................................................................
Patch Set 11: (1 comment) oops, missed one extra nit in previous reviews. Otherwise looks good! http://gerrit.cloudera.org:8080/#/c/4450/11/src/kudu/master/master-path-handlers.cc File src/kudu/master/master-path-handlers.cc: PS11, Line 85: nit: an extra indent. It would be nice to have lines 85, 86, and 87 to be aligned with the first parameter after the opening parenthesis (i.e. like it was in the original code). I somehow missed that in earlier reviews, I apologize. -- To view, visit http://gerrit.cloudera.org:8080/4450 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I479fad5c2db61949f7d67bde7901e7a59c60b786 Gerrit-PatchSet: 11 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Ninad Shringarpure <ni...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes