[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-31 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: consensus: Get rid of ReplicaState class .. consensus: Get rid of ReplicaState class Merges the logic in ReplicaState into the RaftConsensus class.

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-31 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: consensus: Get rid of ReplicaState class .. Patch Set 6: Verified+1 Overriding Spark test cleanup issue: org.apache.kudu.client.NonRecoverableException: Table

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: consensus: Get rid of ReplicaState class .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7007 To unsubscribe, visit

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: consensus: Get rid of ReplicaState class .. Patch Set 6: I should mention that the test I looped 500x was raft_consensus-itest. -- To view, visit

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: consensus: Get rid of ReplicaState class .. Patch Set 6: This version of the patch was run on dist-test for 500 runs under stress (8 cpus) on dist-test. Here are

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-30 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7007 to look at the new patch set (#6). Change subject: consensus: Get rid of ReplicaState class .. consensus: Get rid

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: consensus: Get rid of ReplicaState class .. Patch Set 5: (7 comments) http://gerrit.cloudera.org:8080/#/c/7007/5//COMMIT_MSG Commit Message: Line 12: There are

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: consensus: Get rid of ReplicaState class .. Patch Set 5: (7 comments) http://gerrit.cloudera.org:8080/#/c/7007/5//COMMIT_MSG Commit Message: Line 12: There are

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-27 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7007 to look at the new patch set (#4). Change subject: consensus: Get rid of ReplicaState class .. consensus: Get rid

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-27 Thread Mike Percy (Code Review)
Hello David Ribeiro Alves, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7007 to look at the new patch set (#3). Change subject: consensus: Get rid of ReplicaState class

[kudu-CR] consensus: Get rid of ReplicaState class

2017-05-27 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7007 to look at the new patch set (#2). Change subject: consensus: Get rid of ReplicaState class .. consensus: Get rid