Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16642
to look at the new patch set (#2).
Change subject: [tools] mention whitespace as a separator for variadic args
......................................................................
[tools] mention whitespace as a separator for variadic args
I witnessed a case when people try to use a comma as separators for
a variadic argument in a kudu CLI command. As it turned out, the
help/usage message doesn't state clearly that multiple items in a
variadic argument should be separated by a whitespace. This changelist
updates the description of existing variadic arguments to explicitly
mention that the whitespace is the expected separator there.
This patch does not contain any functional changes.
Change-Id: I3ef979bd17655b5d46e1adb2a2f22923353ac435
---
M src/kudu/tools/tool_action_diagnose.cc
M src/kudu/tools/tool_action_local_replica.cc
M src/kudu/tools/tool_action_remote_replica.cc
3 files changed, 52 insertions(+), 46 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/42/16642/2
--
To view, visit http://gerrit.cloudera.org:8080/16642
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3ef979bd17655b5d46e1adb2a2f22923353ac435
Gerrit-Change-Number: 16642
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
