[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures .. Patch Set 3: (1 comment)

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-29 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures .. Patch Set 3: (7 comments)

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-29 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures .. Patch Set 3: I would also prefer to see these split into several individual patches --

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-29 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Mike Percy, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5111 to look at the new patch set (#3). Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-29 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures .. Patch Set 2: (7 comments)

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures .. Patch Set 2: (4 comments) can you please address the tidy bot nits? I know most

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-18 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5111 to look at the new patch set (#2). Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-16 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures .. Patch Set 1: (1 comment) TFTR David, > (1 comment) > > would it be possible to

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures .. Patch Set 1: (1 comment) would it be possible to split this patch into several

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-16 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5111 to review the following change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures