[GitHub] [lucene] jpountz commented on issue #12140: LRUQueryCache disabled for indices with more than 33 segments

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

jpountz commented on issue #12140:
URL: https://github.com/apache/lucene/issues/12140#issuecomment-1425791875

   Ohhh this is an interesting corner-case for this heuristic. My first reaction was that we could have an exception for segments that reached the maximum tier, but it feels potentially dangerous to iterate over the set of segments to figure out the size of the maximum tier every time we need to make a caching decision. Another idea could be to cache on segments whose number of docs is greater than the average number of docs of segments in the index. This condition would generally be more selective than the 3% threshold, except in the case when there are many segments at the maximum tier.


-- 
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