You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2016/11/23 03:38:07 UTC

[kudu-CR] [java] Reuse snapshot scan timestamp across tablets

Hello Kudu Jenkins,

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

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

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

Change subject: [java] Reuse snapshot scan timestamp across tablets
......................................................................

[java] Reuse snapshot scan timestamp across tablets

KUDU-1189 On reads at a snapshot that touch multiple tablets,
without the user setting a timestamp, use the timestamp from the first
server for following scans

For a READ_AT_SNAPSHOT scan operation with no snapshot timestamp
specified, store the snapshot timestamp returned from the first tablet
server into the scan configuration object. Then reuse it when
continuing the scan on other tablet servers operations performed at
other tablet servers.

Added corresponding unit test as well.

Change-Id: I7207672f7b0cf1307bfa861bda3291b278618016
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestScannerMultiTablet.java
2 files changed, 80 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/88/5188/4
-- 
To view, visit http://gerrit.cloudera.org:8080/5188
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7207672f7b0cf1307bfa861bda3291b278618016
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins