HSEARCH-5010 Disable skipping in comparators

[marko-bekhta]

https://hibernate.atlassian.net/browse/HSEARCH-5010

I can only reproduce the test failure from time to time... the problem is caused by this change:
marko-bekhta/lucene@d910990#diff-79c6a57519ecd1ef504629e62e13d17859a4ffedc58f4602e583ce758a15adc8R288-R295

Since our RawNumericDocValues throws exceptions, the code in the mentioned block just fails. When I've tried to "implement" the two methods throwing the exception -- instead of a failed test because of the exception, the order was incorrect. But again, the issue cannot be reproduced consistently ... 😔

[yrodiere]

LGTM, please port to the lucene9 branch as well :)

But again, the issue cannot be reproduced consistently ... 😔

This doesn't worry me too much, as skipping probably is only possible when documents are distributed a specific way across fragments, and that distribution is notably unpredictable (it depends on the order of indexing operations coming from multiple threads, I think).

Since our RawNumericDocValues throws exceptions, the code in the mentioned block just fails

I think we'll want to solve this eventually, because skipping most likely enables better performance.

I created https://hibernate.atlassian.net/browse/HSEARCH-5014, could you please add the stracktrace there in a comment?

[marko-bekhta]

Thanks for taking a look 😃. Rebased and pushed both (lucene9/lucene10) branches. Closing this draft.