[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1735. Fix crash when aborting a skipped config change round .. KUDU-1735. Fix crash when aborting a skipped config change round This fixes a crash se

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-08 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1735. Fix crash when aborting a skipped config change round .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4916 To unsubscrib

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-08 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4916 to look at the new patch set (#4). Change subject: KUDU-1735. Fix crash when aborting a skipped config change round ...

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1735. Fix crash when aborting a skipped config change round .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4916/2/src/kudu/consensus/raft_c

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1735. Fix crash when aborting a skipped config change round .. Patch Set 3: Code-Review+2 (2 comments) Just nits. ignore/don't ignore, you choose :)

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1735. Fix crash when aborting a skipped config change round .. Patch Set 3: Code-Review+2 (1 comment) Added a nit but the change looks good http://gerrit.clo

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1735. Fix crash when aborting a skipped config change round .. Patch Set 3: This looks pretty good to me. I want to do one more pass on it tomorrow. -- To vi

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1735. Fix crash when aborting a skipped config change round .. Patch Set 3: This is ready for a review (ping Mike/David) -- To view, visit http://gerrit.clo

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-04 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4916 to look at the new patch set (#3). Change subject: KUDU-1735. Fix crash when aborting a skipped config change round

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-02 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4916 to look at the new patch set (#2). Change subject: KUDU-1735. Fix crash when aborting a skipped config change round ...