jbewing commented on PR #5373: URL: https://github.com/apache/hbase/pull/5373#issuecomment-1802083452
The functionality change of https://github.com/apache/hbase/pull/5373/commits/075225e77454f57780f23a83fb2f72c0d20c339e (upgrading from a non-hinted seekToPreviousRow to a hinted seekToPrevious after a single iteration that fails due to skipping kvs) resulted in some persistent test failures in [TestHRegion#testReverseScanner_StackOverflow](https://github.com/apache/hbase/blob/954a1f8fc392d19056cf11b896a188cda462c43b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java#L6404-L6408) due to an infinite recursion 😅. This test was added as part of the remediation for [HBase-14497](https://issues.apache.org/jira/browse/HBASE-14497). I'm going to look at trying to solve this, but for now I just reverted the functionality change as not upgrading from a non-hinted seek to a hinted seek. -- 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