dweiss opened a new pull request, #15493: URL: https://github.com/apache/lucene/pull/15493
Fixes #15309. I've removed the public, linear, remove() method from PriorityQueue. I also made some methods protected which - in case somebody really wants to - can be used to implement the linear remove() in the way it was implemented before. This is actually also the workaround I used for DiversifiedTopDocsCollector. Non-linear remove(element) on a PriorityQueue is not an easy thing to do without somehow tracking positions on the heap, which would make everything slower. Let's keep the essentials that are fast instead. This patch makes certain PQ members protected (size, upHeap, downHeap). I think it makes sense to do so because we already expose the heap buffer and allow subclassing. Seems like the above are necessary ingredients for implementing something reasonable in a subclass. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
