[kudu-CR] consensus: separate storage from ConsensusMetadata

2018-06-25 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10649 ) Change subject: consensus: separate storage from ConsensusMetadata .. Patch Set 4: (5 comments) I guess andrew's point is that

[kudu-CR] consensus: separate storage from ConsensusMetadata

2018-06-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10649 ) Change subject: consensus: separate storage from ConsensusMetadata .. Patch Set 4: Code-Review+1 (3 comments) Looks pretty good to me.

[kudu-CR] consensus: separate storage from ConsensusMetadata

2018-06-19 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10649 ) Change subject: consensus: separate storage from ConsensusMetadata .. Patch Set 4: (3 comments)

[kudu-CR] consensus: separate storage from ConsensusMetadata

2018-06-18 Thread Andrew Wong (Code Review)
Hello Tidy Bot, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10649 to look at the new patch set (#3). Change subject: consensus: separate storage from ConsensusMetadata

[kudu-CR] consensus: separate storage from ConsensusMetadata

2018-06-18 Thread Andrew Wong (Code Review)
Hello Tidy Bot, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10649 to look at the new patch set (#2). Change subject: consensus: separate storage from ConsensusMetadata