David Ribeiro Alves has posted comments on this change. Change subject: [java] KUDU-1679 Propagate timestamps for scans ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5248/1/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java File java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java: PS1, Line 214: protected synchronized long getLastPropagatedTimestamp() > That's an interesting observation. the whole point of the lpt is that users can use message passing to get linearizabiilty so it should always be user visible. We're just making use of it ourselves for emulating a single client from many clients scenario. BTW Get/SetLatestObservedTimestamp are public methods on the c++ client -- To view, visit http://gerrit.cloudera.org:8080/5248 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I84d45ba395f2a3fc6b54591f4a45bb4f10435910 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes