Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19650 )
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 initialize 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 test case to ensure we can get the same keys from the bootstrap process at startup. Change-Id: I1b7ed75bdf7a2ff0e16c2670f1a6f9819ee8e8d3 Reviewed-on: http://gerrit.cloudera.org:8080/19650 Tested-by: Kudu Jenkins Reviewed-by: Yuqi Du <[email protected]> Reviewed-by: Yingchun Lai <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- 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, 119 insertions(+), 5 deletions(-) Approvals: Kudu Jenkins: Verified Yuqi Du: Looks good to me, but someone else must approve Yingchun Lai: Looks good to me, but someone else must approve Alexey Serbin: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I1b7ed75bdf7a2ff0e16c2670f1a6f9819ee8e8d3 Gerrit-Change-Number: 19650 Gerrit-PatchSet: 11 Gerrit-Owner: KeDeng <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]>
