[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [tools] Implement a manual leader_step_down for a tablet .. [tools] Implement a manual leader_step_down for a tablet This change introduces a leader_ste

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 13: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4533 To unsubscribe, v

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-07 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/4533/12/src/kudu/tools/tool_action_table

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-07 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#13). Change subject: [tools] Implement a manual leader_step_down for a tablet

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 12: (1 comment) > > > Patch Set 11: Patch Set 10 was rebased > > > > Could you also add

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-07 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 12: > > Patch Set 11: Patch Set 10 was rebased > > Could you also add coverage for the new

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-07 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#12). Change subject: [tools] Implement a manual leader_step_down for a tablet

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 11: > Patch Set 11: Patch Set 10 was rebased Could you also add coverage for the new sub-c

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-03 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#10). Change subject: [tools] Implement a manual leader_step_down for a tablet

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-03 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 9: (5 comments) > (5 comments) > > There is one extra instance of "master_addresses" stri

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-10-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 9: (5 comments) There is one extra instance of "master_addresses" string literal and corr

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4533 To unsubscribe, visit

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 9: (6 comments) http://gerrit.cloudera.org:8080/#/c/4533/6/src/kudu/tools/kudu-admin-test.cc

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#9). Change subject: [tools] Implement a manual leader_step_down for a tablet ..

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 8: (5 comments) http://gerrit.cloudera.org:8080/#/c/4533/8/src/kudu/tools/kudu-admin-test.

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/4533/6/src/kudu/tools/kudu-admin-test.cc F

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#8). Change subject: [tools] Implement a manual leader_step_down for a tablet ..

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/4533/6/src/kudu/tools/kudu-admin-test.cc

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#7). Change subject: [tools] Implement a manual leader_step_down for a tablet ..

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/4533/6/src/kudu/tools/kudu-admin-test.cc

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-30 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#6). Change subject: [tools] Implement a manual leader_step_down for a tablet ..

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-29 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#5). Change subject: [tools] Implement a manual leader_step_down for a tablet ..

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 4: Please rebase this patch before pushing another revision. -- To view, visit http://gerri

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-29 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#4). Change subject: [tools] Implement a manual leader_step_down for a tablet ..

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-29 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/4533/1/src/kudu/tools/kudu-admin-test.cc

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/4533/1/src/kudu/tools/kudu-admin-test.cc

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-29 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 3: (9 comments) TFTR Adar, pls see responses inline, also pls see why my jenkins are still

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-29 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#3). Change subject: [tools] Implement a manual leader_step_down for a tablet ..

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 2: (7 comments) http://gerrit.cloudera.org:8080/#/c/4533/1/src/kudu/tools/kudu-admin-test.cc

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-28 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 1: (13 comments) http://gerrit.cloudera.org:8080/#/c/4533/1//COMMIT_MSG Commit Message: PS1

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-28 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4533 to look at the new patch set (#2). Change subject: [tools] Implement a manual leader_step_down for a tablet ..

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [tools] Implement a manual leader_step_down for a tablet .. Patch Set 1: (17 comments) Please also update kudu-tool-test's help tests with the new action. http://

[kudu-CR] [tools] Implement a manual leader step down for a tablet

2016-09-23 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4533 to review the following change. Change subject: [tools] Implement a manual leader_step_down for a tablet .