[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [tools] added insert-generated-rows into kudu tools .. [tools] added insert-generated-rows into kudu tools The insert-generated-rows tool has been merge

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 22: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4412 To unsubscribe, visit htt

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#22). Change subject: [tools] added insert-generated-rows into kudu tools

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 21: (1 comment) http://gerrit.cloudera.org:8080/#/c/4412/21/src/kudu/tools/tool_action_test.cc

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 21: (1 comment) http://gerrit.cloudera.org:8080/#/c/4412/21/src/kudu/tools/tool_action_test.cc Fil

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 20: (2 comments) http://gerrit.cloudera.org:8080/#/c/4412/20/src/kudu/tools/tool_action_test.cc

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#21). Change subject: [tools] added insert-generated-rows into kudu tools

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 20: (2 comments) http://gerrit.cloudera.org:8080/#/c/4412/20/src/kudu/tools/tool_action_test.cc Fi

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#19). Change subject: [tools] added insert-generated-rows into kudu tools

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 18: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4412 To unsubscribe, visit htt

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 16: (3 comments) http://gerrit.cloudera.org:8080/#/c/4412/16/src/kudu/tools/kudu-tool-test.cc F

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#18). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 17: (3 comments) http://gerrit.cloudera.org:8080/#/c/4412/16/src/kudu/tools/kudu-tool-test.cc File

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 16: (7 comments) http://gerrit.cloudera.org:8080/#/c/4412/16/src/kudu/tools/kudu-tool-test.cc F

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#17). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 16: (7 comments) http://gerrit.cloudera.org:8080/#/c/4412/16/src/kudu/tools/kudu-tool-test.cc File

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#16). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/4412/15/src/kudu/tools/kudu-test.cc File src/k

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 15: (5 comments) http://gerrit.cloudera.org:8080/#/c/4412/15/src/kudu/tools/CMakeLists.txt File

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 15: (5 comments) http://gerrit.cloudera.org:8080/#/c/4412/15/src/kudu/tools/CMakeLists.txt File sr

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#15). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#14). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 13: (2 comments) http://gerrit.cloudera.org:8080/#/c/4412/12/src/kudu/tools/tool_action_test.cc F

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 12: (20 comments) http://gerrit.cloudera.org:8080/#/c/4412/11/src/kudu/tools/tool_action_test.c

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-27 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#13). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-27 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 12: (21 comments) http://gerrit.cloudera.org:8080/#/c/4412/12/src/kudu/tools/tool_action_test.cc

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-27 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 12: (1 comment) Looks good, though see my previous comment (PS11) about adding a few tests. http:

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-27 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#12). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-27 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/4412/11/src/kudu/tools/tool_action_test.cc

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-27 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 11: (2 comments) http://gerrit.cloudera.org:8080/#/c/4412/11/src/kudu/tools/tool_action_test.cc Fi

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-27 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 11: (26 comments) Will post updated code shortly. http://gerrit.cloudera.org:8080/#/c/4412/11/

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] added insert-generated-rows into kudu tools .. Patch Set 11: (26 comments) Can you add a couple things to kudu-tool-test? 1. Modify the help tests to incor

[kudu-CR] [tools] added insert-generated-rows into kudu tools

2016-09-26 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4412 to look at the new patch set (#11). Change subject: [tools] added insert-generated-rows into kudu tools .. [tools]