Hao Hao 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
Yes, since there is no write from _this client_, there is no guarantee on how 
many rows we can get back in this read mode. The test is mainly to  ensure the 
scan can return without an error as described in KUDU-2415.



--
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 <hao....@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Sun, 27 Jan 2019 05:50:34 +0000
Gerrit-HasComments: Yes

Reply via email to