Hello Alexey Serbin, Yuqi Du, Yingchun Lai, Kudu Jenkins, Wang Xixu,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19650
to look at the new patch set (#4).
Change subject: [rowset_metadata] update min/max encoded keys during
bootstrapping
......................................................................
[rowset_metadata] update min/max encoded keys during bootstrapping
We can cache the encoded min/max keys in rowset metadata to help
tablets to bootstrap without having to fully initializing the CFileReaders
for the key columns of each rowset. However, we can only update the
min/max encoded keys during insert or update operations right now.
This means that if we didn't save the min/max encoded key before,
we won't have the opportunity to use this feature to accelerate bootstrap
in the future.
Considering that we will read the min/max key values during the bootstrap
process, we can set the min/max encoded keys in the metadata at the same
time.
I added a small case to ensure we can get the same keys from the bootstrap
process at startup.
Change-Id: I1b7ed75bdf7a2ff0e16c2670f1a6f9819ee8e8d3
---
M src/kudu/tablet/cfile_set.cc
M src/kudu/tablet/cfile_set.h
M src/kudu/tablet/diskrowset.h
M src/kudu/tablet/tablet.h
M src/kudu/tserver/tablet_server-test.cc
5 files changed, 116 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/19650/4
--
To view, visit http://gerrit.cloudera.org:8080/19650
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1b7ed75bdf7a2ff0e16c2670f1a6f9819ee8e8d3
Gerrit-Change-Number: 19650
Gerrit-PatchSet: 4
Gerrit-Owner: KeDeng <kdeng...@gmail.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: KeDeng <kdeng...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Yuqi Du <shenxingwuy...@gmail.com>
