[ https://issues.apache.org/jira/browse/LUCENE-9287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17065790#comment-17065790 ]
Adrien Grand commented on LUCENE-9287: -------------------------------------- The change makes sense to me but I don't understand the benchmark, does it mean that the query takes 40ms once cached? Or is it the time it takes to cache the query? > Never cache DocValuesFieldExistsQuery > ------------------------------------- > > Key: LUCENE-9287 > URL: https://issues.apache.org/jira/browse/LUCENE-9287 > Project: Lucene - Core > Issue Type: Improvement > Reporter: Ignacio Vera > Priority: Minor > Time Spent: 10m > Remaining Estimate: 0h > > Currently {{DocValuesFieldExistsQuery}} is cached if there are no DV updates > on the field in question. This query is pretty efficient and some experiments > show that caching the result might be much more expensive than the benefits > we get from caching the result. > For example, indexing 10 million documents with long values and then > executing this query: > * without caching it takes around 1ms to run. > * with caching it takes around 40ms to run and the size of the cache is > around 1MB (1212792 bytes). > I wonder if we should never cache this query as it seems the overhead of > caching the results is not worthy. > -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org