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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
