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 (#5). 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, 104 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/19650/5 -- 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: 5 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>