OldRepositoryAccessIT.testOldRepoAccess fails on Lucene Snapshot about elasticsearch HOT 3 CLOSED

Pinging @elastic/es-search (Team:Search)

from elasticsearch.

Its a blocker for Lucene Snapshot, but I don't know how to communicate that via labels. And I don't want folks to think its a blocker for anything else.

So, low-risk for now.

from elasticsearch.

This is due to a recent Lucene change, however I don't think that the Lucene change is the problem but that it just makes an existing problem visible now that the numeric comparator pull a point tree eagerly in its constructor rather than lazily. We were not seeing the problem before because we're searching over less than IndexSearcher#TOTAL_HITS_THRESHOLD so the dynamic pruning logic never kicks in.

Lucene's sorting logic assumes that if there are points, then they can be used for dynamic pruning. But this doesn't work since the old file format only supports metadata operations. A fix could consist of hiding points, e.g. by disabling them in field infos, but then the index wouldn't be able to use them for can_match phases.

from elasticsearch.