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
