Alexey Serbin has posted comments on this change. Change subject: KUDU-1189 scans: reuse snapshot timestamp when not set ......................................................................
Patch Set 1: > (1 comment) > > Can you add tests for this? Thank you for the review! Yes, I'm working on the test right now. The idea is to put the test first, but have it disabled (since it's supposed to fail without this fix), and then enable it with the next version of this patch. I sent this to collect initial feedback since my knowledge of the Java client is limited. -- To view, visit http://gerrit.cloudera.org:8080/5188 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7207672f7b0cf1307bfa861bda3291b278618016 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No
