Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12278 )

Change subject: [java] enable regression test for KUDU-2415
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12278/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestScannerMultiTablet.java
File 
java/kudu-client/src/test/java/org/apache/kudu/client/TestScannerMultiTablet.java:

http://gerrit.cloudera.org:8080/#/c/12278/1/java/kudu-client/src/test/java/org/apache/kudu/client/TestScannerMultiTablet.java@288
PS1, Line 288: // Since there isn't any write performed from the client, the 
count
             :     // should range from [0, 9].
Not sure I fully understand; does this mean to say that because there have been 
no writes, there is no guarantee on the rows that we get back in 
READ_YOUR_WRITES mode? Otherwise, why wouldn't it be exactly 9?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9532a435c3b4cf2286630b6809ffa2c8ae1a4acd
Gerrit-Change-Number: 12278
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Sat, 26 Jan 2019 21:58:37 +0000
Gerrit-HasComments: Yes

Reply via email to