Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13667 )
Change subject: KUDU-2867. Optimize delta timestamp decoding ...................................................................... KUDU-2867. Optimize delta timestamp decoding When using hybrid clock, all timestamps require a full 8-byte varint to be encoded. So, we can inline this fast path in Timestamp::DecodeFrom to avoid the more complicated/lengthy GetMemcmpableVarint code. Change-Id: I1665e0aaed874cf4d47ecf5e467246a9505b0bd1 Reviewed-on: http://gerrit.cloudera.org:8080/13667 Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <granthe...@apache.org> Reviewed-by: Alexey Serbin <aser...@cloudera.com> --- M src/kudu/common/timestamp.cc M src/kudu/common/timestamp.h 2 files changed, 20 insertions(+), 6 deletions(-) Approvals: Kudu Jenkins: Verified Grant Henke: Looks good to me, but someone else must approve Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13667 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1665e0aaed874cf4d47ecf5e467246a9505b0bd1 Gerrit-Change-Number: 13667 Gerrit-PatchSet: 3 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <t...@apache.org>