Apache9 commented on a change in pull request #3639:
URL: https://github.com/apache/hbase/pull/3639#discussion_r697811738
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
##########
@@ -737,7 +737,10 @@ public boolean next(List<Cell> outResult, ScannerContext
scannerContext) throws
case SEEK_NEXT_USING_HINT:
Cell nextKV = matcher.getNextKeyHint(cell);
- if (nextKV != null && comparator.compare(nextKV, cell) > 0) {
+ if (nextKV != null &&
+ ((!scan.isReversed() && comparator.compare(nextKV, cell) > 0)
Review comment:
OK, thank you for making it more clear.
On the broken PR state, it is because this issue
https://issues.apache.org/jira/browse/SUREFIRE-1821
We enable flaky rerun feature and once there are some flaky tests, the xml
report will be broken and fail the report generation stage of the jenkins job,
which will finally lead a broken state here.
Still trying to solve the problem but not a blocker for merging here.
--
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]
