Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4569 to look at the new patch set (#6). Change subject: [c++ client] notes for timestamp-related methods ...................................................................... [c++ client] notes for timestamp-related methods Added notes on anticipated changes on methods related to getting and setting hybrid timestamps while performing READ_AT_SNAPSHOT scan operations: * KuduClient::GetLatestObservedTimestamp() * KuduClient::SetLatestObservedTimestamp() * KuduScanner::SetSnapshotRaw() Some changes are anticipated there in the context of KUDU-611. In short, the timestamp might be replaced with some opaque type (e.g., a sequence of bytes). Also, added code sample to provide an idea how to use value returned by KuduClient::GetLatestObservedTimestamp() to set appropriate snapshot timestamp via KuduScanner::SetSnapshotRaw(). This addresses the following JIRA issues: KUDU-1661 Mark KuduClient::GetLatestObservedTimestamp() as experimental KUDU-1663 Clean-up in-code documentation for KuduScanner::ReadMode and KuduClient::GetLatestObservedTimestamp Change-Id: I6c45b797fa459ac9d214bb2612fd797f7a1eea45 --- M src/kudu/client/client.h 1 file changed, 43 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/4569/6 -- To view, visit http://gerrit.cloudera.org:8080/4569 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I6c45b797fa459ac9d214bb2612fd797f7a1eea45 Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>