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

To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to