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 6:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/4450/6/src/kudu/master/master-path-handlers.cc
File src/kudu/master/master-path-handlers.cc:

PS6, Line 109:                              
formatting nit: the second and third parameter lines are not aligned with the 
first.  Consider making them aligned.


PS6, Line 111:       
nit: by style guide, the indent is 2 spaces.


Line 121: 
nit: an extra line


-- 
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: 6
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

Reply via email to