Adar Dembo has posted comments on this change.

Change subject: tool: Replace Kudu replica with tablet replica in help text
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/5310/1/src/kudu/tools/kudu-tool-test.cc
File src/kudu/tools/kudu-tool-test.cc:

Line 316:     const vector<string> kLocalReplicaModeRegexes = {
Nit: rename to kLocalReplicaCmeta (since it's the cmeta submode of the local 
replica mode).


http://gerrit.cloudera.org:8080/#/c/5310/1/src/kudu/tools/tool_action_local_replica.cc
File src/kudu/tools/tool_action_local_replica.cc:

Line 699:       .Description("Print all replica peer UUIDs found in a "
Not "tablet replica peer UUIDs" here?


-- 
To view, visit http://gerrit.cloudera.org:8080/5310
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iec236fb145cd4b9c7a5027898f9beb9262f5893a
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to