PDavid opened a new pull request, #6482:
URL: https://github.com/apache/hbase/pull/6482

   This is the backport of https://github.com/apache/hbase/pull/6457 to 
`branch-2`
   
   Changes:
   
   - Reproduced the issue with a mini cluster test.
   - Corrected reverse unit tests in `TestFuzzyRowFilter` as some expectations 
were not correct.
   - Fixed hint returned by `FuzzyRowFilter.getNextForFuzzyRule()` in the 
reverse case:
     - Before we only trimmed the trailing zeros from the hint in the forward 
scan case. From now on we also do this in the reverse case.
       - Also extracted `PrefixFilter.increaseLastNonMaxByte()` method to 
`PrivateCellUtil` so that it can be reused in `FuzzyRowFilter`.
     - When the hint should contain a `0xFF`. As a start, when we have a 
reverse scan, we not only need trailing `0xff`'s instead of trailing `0x00`'s, 
but it seems we always need `0xff`'s instead of the `0x00`'s.
   - Made some small improvements to `FuzzyRowFilter` which IntelliJ IDEA 
suggested (final fields, simplifications).
   


-- 
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: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to