sunhelly commented on code in PR #4536:
URL: https://github.com/apache/hbase/pull/4536#discussion_r1118381472
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java:
##########
@@ -768,7 +768,11 @@ public boolean next(List<Cell> outResult, ScannerContext
scannerContext) throws
// One last chance to break due to size limit. The INCLUDE* cases
above already check
// limit and continue. For the various filtered cases, we need to
check because block
// size limit may have been exceeded even if we don't add cells to
result list.
- if (scannerContext.checkSizeLimit(LimitScope.BETWEEN_CELLS)) {
+ // And when reaching the heartbeat cells, try to return from the loop.
+ if (
+ scannerContext.checkSizeLimit(LimitScope.BETWEEN_CELLS)
+ || kvsScanned % cellsPerHeartbeatCheck == 0
Review Comment:
We have discussed about the feature applying to compaction scan or user scan
in https://issues.apache.org/jira/browse/HBASE-25709. Actually, we want the
scans not stuck/loop in skip cells, and both the user and the compaction scans
should return as soon as possible.
--
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]
