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 ...................................................................... consensus: separate storage from ConsensusMetadata This patch removes the underlying storage mechanisms from the ConsensusMetadata class and moves it into the ConsensusMetadataManager. Instead, the cmeta manager will now coordinate flushing of the cmeta, as it already manages their loading and creation. This move will make changes to the cmeta's on-disk storage possible without changing the interfaces and implementation of the ConsensusMetadata class itself. Some of the semantics are changed for the ConsensusMetadataManager. Namely, because ConsensusMetadata's Create(), Load(), etc., now depend on the manager, which maintains a cache for the metadata, the aforementioned functions will all refer to the same ConsensusMetadata object. Change-Id: I2f17123e281789976df1c436fe8116a4d4b48923 --- M src/kudu/consensus/consensus_meta-test.cc M src/kudu/consensus/consensus_meta.cc M src/kudu/consensus/consensus_meta.h M src/kudu/consensus/consensus_meta_manager-stress-test.cc M src/kudu/consensus/consensus_meta_manager.cc M src/kudu/consensus/consensus_meta_manager.h M src/kudu/tserver/ts_tablet_manager.cc 7 files changed, 260 insertions(+), 128 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/10649/2 -- 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: newpatchset Gerrit-Change-Id: I2f17123e281789976df1c436fe8116a4d4b48923 Gerrit-Change-Number: 10649 Gerrit-PatchSet: 2 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot