[kudu-CR] tool: better handling for positional arguments

2016-08-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: tool: better handling for positional arguments .. tool: better handling for positional arguments This patch modifies Action to describe all of its

[kudu-CR] tool: better handling for positional arguments

2016-08-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4013 To unsubscribe, visit

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4013/3/src/kudu/tools/tool_action.cc File

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2985/ -- To view, visit

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4013 to look at the new patch set (#4). Change subject: tool: better handling for positional arguments

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4013/3/src/kudu/tools/tool_action.cc File

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2981/ -- To view, visit

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Adar Dembo (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4013 to look at the new patch set (#3). Change subject: tool: better handling for positional arguments ..

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4013/1/src/kudu/tools/tool_action.h File

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Adar Dembo (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4013 to look at the new patch set (#2). Change subject: tool: better handling for positional arguments ..

[kudu-CR] tool: better handling for positional arguments

2016-08-18 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2974/ -- To view, visit

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4013/1/src/kudu/tools/tool_action.h File

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: (4 comments) Sure wish there was some test coverage on these tools...

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: > (5 comments) > > There is a Go library which allows to implement rich CLI tools. If >

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: Code-Review+1 (5 comments) There is a Go library which allows to implement rich CLI tools. If