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:24:34 UTC

[kudu-CR] KUDU-1189 scans: reuse snapshot timestamp when not set (Java)

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 (#2).

Change subject: KUDU-1189 scans: reuse snapshot timestamp when not set (Java)
......................................................................

KUDU-1189 scans: reuse snapshot timestamp when not set (Java)

This is Java-side part of 41fa38547f640caeeca95a09cf1e32da8c56872b.

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.

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, 81 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/88/5188/2
-- 
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: 2
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