Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13435 )
Change subject: [KUDU-2825]add a column named role in 'kudu master list' ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/13435/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/13435/2//COMMIT_MSG@7 PS2, Line 7: [KUDU-2825]add a column named role in 'kudu master list' > nit: add a space after the closing bracket I'd also reformat as: KUDU-2825: add a column named role in 'kudu master list' We typically use square brackets to describe functional areas, like [java], [tablet], or [master]. -- To view, visit http://gerrit.cloudera.org:8080/13435 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I29da5024ec59d3fb1d576247b2f0f7af0e7968dc Gerrit-Change-Number: 13435 Gerrit-PatchSet: 2 Gerrit-Owner: GuangchaoDeng <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: helifu <[email protected]> Gerrit-Comment-Date: Mon, 27 May 2019 22:11:49 +0000 Gerrit-HasComments: Yes
