Adar Dembo has posted comments on this change.

Change subject: KUDU-1343. [java client] Scan sequence id is never reset 
between tablets
......................................................................


Patch Set 2:

Fix looks good, but why is the new test an integration test? If the issue is 
that writing and scanning 30000 rows takes too long, you could do what Todd 
said and set the scan batch size low enough that you'll get the multi-batch 
behavior you want with only a few rows.

-- 
To view, visit http://gerrit.cloudera.org:8080/2301
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee05caa9157288e21475c64ade6a8dc7f6609665
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to