bbeaudreault commented on a change in pull request #3532:
URL: https://github.com/apache/hbase/pull/3532#discussion_r686281797



##########
File path: hbase-client/src/main/java/org/apache/hadoop/hbase/client/Get.java
##########
@@ -339,6 +340,21 @@ public Get setFilter(Filter filter) {
     return this;
   }
 
+  /**
+   * Set the maximum result size. The default is -1; this means that no 
specific
+   * maximum result size will be set for this Get.
+   *
+   * If set to a value greater than zero, the server may respond with a Result 
where
+   * {@link Result#mayHaveMoreCellsInRow()} is true. The user is required to 
handle
+   * this case.

Review comment:
       In terms of that last sentence, maybe it's better to not support 
stitching for Gets. Instead people should rewrite these large Gets as Scans or 
add filters like above. Stitching obviously increases the latency and that 
could be very misleading for Gets. Multigets even worse (and harder to 
implement)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to