Hello Kudu Jenkins, Grant Henke,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/13667

to look at the new patch set (#2).

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
---
M src/kudu/common/timestamp.cc
M src/kudu/common/timestamp.h
2 files changed, 20 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/13667/2
--
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: newpatchset
Gerrit-Change-Id: I1665e0aaed874cf4d47ecf5e467246a9505b0bd1
Gerrit-Change-Number: 13667
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to