kkewwei commented on issue #16495: URL: https://github.com/apache/lucene/issues/16495#issuecomment-5248959739
@sgup432 Thanks, that makes sense to me. I agree this should be an opt-in feature. My current thinking: 1. Expose a public API on `LRUQueryCache` to retrieve a bounded snapshot of recently used unique queries, for example something like `cachedQueriesSnapshot(int maxQueries)`. Internally this could be derived from the existing `uniqueCacheKeys` / `cachedQueries()` logic. 2. Reuse the existing `IndexWriterConfig#setMergedSegmentWarmer` hook as the trigger point for merged segments. The warmer could then decide whether to prewarm selected queries for the newly created segment. I am not sure if other maintainers have additional thoughts or concerns about the API shape, but I would be happy to continue exploring an implementation based on this direction. ```` -- 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]
