EungsopYoo commented on PR #8360: URL: https://github.com/apache/hbase/pull/8360#issuecomment-4726271245
Thanks for catching this. I reworked the fix so `getNextForFuzzyRule` keeps the existing reverse hint behavior, including the HBASE-28634 adjustment. The progress guard now lives in `RowTracker` / `getNextCellHint`: when a reverse hint does not move before the current row, `FuzzyRowFilter` returns a non-seeking hint and lets `StoreScanner` advance with `heap.next()`. I also added `1115` to `testHBASE28634` so the regression you reproduced is covered, and kept the new `abc -> abb -> aaa` regression test for the same-row hint hang. Pushed commit: 44778a4f8026cb87fce36b0074d7437d43a9ffc3 Verified: - `mvn -pl hbase-server -am -Dtest=TestFuzzyRowFilterEndToEnd#testReverseScanMovesPastSameRowFuzzyHint+testHBASE28634 -DexcludedGroups=slow -Dwarbucks.skip=true test` - changed test file checkstyle -- 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]
