[
https://issues.apache.org/jira/browse/LUCENE-8144?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16351098#comment-16351098
]
Hoss Man commented on LUCENE-8144:
----------------------------------
IIUC: would it be useful to move this to TestLRUQueryCache rather then removing
it outright?
At a glance, it seems kind of important to keep using this in TestLRUQueryCache
to ensure the test behaves reliably regardless of what the default policy might
be (or how the default policy might treat the queries that just happen to be
used in TestLRUQueryCache -- now or down the road)
> Remove QueryCachingPolicy.ALWAYS_CACHE
> --------------------------------------
>
> Key: LUCENE-8144
> URL: https://issues.apache.org/jira/browse/LUCENE-8144
> Project: Lucene - Core
> Issue Type: Task
> Reporter: Adrien Grand
> Priority: Minor
>
> This one is trappy, it looks simple and cool but caching without evidence of
> reuse is usually a bad idea as it removes the ability to skip over non
> interesting documents.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
