kadirozde commented on a change in pull request #1111:
URL: https://github.com/apache/phoenix/pull/1111#discussion_r564860842
##########
File path: phoenix-core/src/main/java/org/apache/phoenix/util/ScanUtil.java
##########
@@ -1304,21 +1304,20 @@ public static PagedFilter getPhoenixPagedFilter(Scan
scan) {
* each HBase RegionScanner#next() time which is controlled by PagedFilter
is set to 0.3 * SERVER_PAGE_SIZE_MS.
*
*/
- private static long getPageSizeMs(Scan scan) {
+ private static long getPageSizeMs(Scan scan, double factor) {
long pageSizeMs = Long.MAX_VALUE;
byte[] pageSizeMsBytes =
scan.getAttribute(BaseScannerRegionObserver.SERVER_PAGE_SIZE_MS);
if (pageSizeMsBytes != null) {
pageSizeMs = Bytes.toLong(pageSizeMsBytes);
+ pageSizeMs = (long) (pageSizeMs * factor);
}
return pageSizeMs;
}
- public static long getPageSizeMsForRegionScanner(Scan scan) {
- return (long) (getPageSizeMs(scan) * 0.6);
- }
+ public static long getPageSizeMsForRegionScanner(Scan scan) { return
getPageSizeMs(scan, 0.6); }
Review comment:
The comments are just right there before getPageSizeMs()
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
