Apache9 commented on PR #7517: URL: https://github.com/apache/hbase/pull/7517#issuecomment-3620522348
Checked the code, the only place where we set keepProgress to true is in RegionScannerImpl.populateResult, where we want to keep progress across families, i.e, different StoreScanners. This is the keepProgress designed to do. The ScannerContext is not designed to enforce quota, so I think the usage in HBASE-27558 is incorrect, at least, it should not mess up with other limits. Let me think what is the better way to fix it, at least, we should change the code added in HBASE-27558, not the code in other places... Thanks. -- 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]
