[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-23 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: KUDU-1407: reassign failed tablets .. KUDU-1407: reassign failed tablets Tablets put into the state tablet::FAILED are left until they are manually

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 20: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7440 To unsubscribe, visit

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 19: (1 comment) http://gerrit.cloudera.org:8080/#/c/7440/19/src/kudu/tserver/ts_tablet_manager.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 20: (6 comments) http://gerrit.cloudera.org:8080/#/c/7440/19/src/kudu/consensus/consensus_queue.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-23 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#20). Change subject: KUDU-1407: reassign failed tablets ..

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 19: (1 comment) http://gerrit.cloudera.org:8080/#/c/7440/19/src/kudu/tserver/ts_tablet_manager.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 19: (1 comment) http://gerrit.cloudera.org:8080/#/c/7440/19/src/kudu/consensus/consensus_queue.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 19: (6 comments) http://gerrit.cloudera.org:8080/#/c/7440/19/src/kudu/consensus/consensus_queue.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 19: The jenkins failure doesn't seem to be related to these changes. This patch is still good for review. -- To

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#19). Change subject: KUDU-1407: reassign failed tablets ..

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 17: (8 comments) http://gerrit.cloudera.org:8080/#/c/7440/17//COMMIT_MSG Commit Message: PS17, Line 30: is added

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#18). Change subject: KUDU-1407: reassign failed tablets ..

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 17: (8 comments) http://gerrit.cloudera.org:8080/#/c/7440/17//COMMIT_MSG Commit Message: PS17, Line 30: is added

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 17: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7440 To unsubscribe, visit

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 17: This patch has been +2'd, but some of the logic around failing tablet replicas overlaps with the tombstoned

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-21 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 15: Just rebased this in my series of start-time failure-handling patches, since it's necessary for

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-18 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#14). Change subject: KUDU-1407: reassign failed tablets ..

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 13: (1 comment) @dralves Opted out of doing the enums thing we discussed offline. I think there are few enough

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-18 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#13). Change subject: KUDU-1407: reassign failed tablets ..

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/7440/12/src/kudu/tserver/ts_tablet_manager.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 12: Code-Review-1 (1 comment) I'm a little suspicious of the build failure; I'm currently running a dist-test to

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 12: (1 comment) Feel free to ignore my feedback if you weren't going to generate a new patch since it's minor.

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 12: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7440 To unsubscribe, visit

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-17 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 11: (3 comments)

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-17 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#12). Change subject: KUDU-1407: reassign failed tablets .. KUDU-1407: reassign

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-17 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 7: (7 comments) http://gerrit.cloudera.org:8080/#/c/7440/7/src/kudu/client/scanner-internal.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-17 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#11). Change subject: KUDU-1407: reassign failed tablets .. KUDU-1407: reassign

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1407: reassign failed tablets .. Patch Set 10: (6 comments) http://gerrit.cloudera.org:8080/#/c/7440/7/src/kudu/client/scanner-internal.cc File

[kudu-CR] KUDU-1407: reassign failed tablets

2017-08-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7440 to look at the new patch set (#9). Change subject: KUDU-1407: reassign failed tablets .. KUDU-1407: reassign