[GitHub] [lucene] jbellis commented on pull request #12255: allocate one NeighborQueue per search for results

["jbellis (via GitHub)" <gi...@apache.org>]

jbellis commented on PR #12255:
URL: https://github.com/apache/lucene/pull/12255#issuecomment-1528882157

   @msokolov wrote earlier,
   
   > I'm not entirely sure why just from inspection, but this seems to have broken some of the backwards-compatibility tests. What it means is this can no longer read indexes written by a prior point release. Maybe consider reverting the change to pre-allocate the NeighborQueue so we can move forward with the change from TreeMap to HashMap?
   
   How can I reproduce this locally?  `gradlew test` reports all successful.


-- 
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-unsubscribe@lucene.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org