jpisaac commented on a change in pull request #1111:
URL: https://github.com/apache/phoenix/pull/1111#discussion_r564843145
##########
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:
What is/should be the thought process in choosing the factor? May be
good to add some comments around it?
----------------------------------------------------------------
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
