Andy Webb
Andy Webb
Thanks @jpountz! I've switched to a new `originalIndex` property - the test passes, I'll check this behaves as expected locally too...
It's working as expected when deployed, but I saw that `org.apache.lucene.search.TestQueryRescorer.testRandom` became unreliable with this change - it failed for about 1 in 8 runs until I updated the test...
I've been exploring constraining `maxValue` so we're more likely to see same-score items each time - e.g. using 50 in `int maxValue = TestUtil.nextInt(random(), 10, 50);` rather than a million...
Just picking this up again (sorry - life, etc!) - I've rebased my branch to pick up recent updates. @jpountz your comment above sounds to me like undoing the change...