David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10649 )
Change subject: WIP consensus: remove env usage from ConsensusMetadata ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/10649/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/10649/1//COMMIT_MSG@7 PS1, Line 7: WIP consensus: remove env usage from ConsensusMetadata this is a bit mistleading, this is more about decoupling ConsensusMetadata from FsManager and pipe (the leftover operation) the operations though ConsensusMetadataManager http://gerrit.cloudera.org:8080/#/c/10649/1/src/kudu/consensus/consensus_meta_manager.h File src/kudu/consensus/consensus_meta_manager.h: http://gerrit.cloudera.org:8080/#/c/10649/1/src/kudu/consensus/consensus_meta_manager.h@52 PS1, Line 52: Status Init() const; docs http://gerrit.cloudera.org:8080/#/c/10649/1/src/kudu/consensus/consensus_meta_manager.h@91 PS1, Line 91: void ClearCache(const std::string& tablet_id); docs -- To view, visit http://gerrit.cloudera.org:8080/10649 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2f17123e281789976df1c436fe8116a4d4b48923 Gerrit-Change-Number: 10649 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Fri, 08 Jun 2018 21:52:41 +0000 Gerrit-HasComments: Yes