[ 
https://issues.apache.org/jira/browse/LUCENE-5114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13710328#comment-13710328
 ] 

David Smiley commented on LUCENE-5114:
--------------------------------------

bq. If you want to "leave" the old method, please please make it final... 
(otherwise, don't do it)

+1

bq. In all cases the API must break, or will only invite bugs.

What bugs?  There wasn't any caching in the first place; it was effectively a 
no-op.

bq. Also, I'd rather on upgrade that expert users using these methods realize 
(because they see the compilation errors) that useCache is/was not doing 
anything ...

I think deprecation warnings are fine.  Using these methods is not so serious 
to warrant compiled extensions being incompatible.

                
> remove boolean useCache param from TermsEnum.seekCeil/Exact
> -----------------------------------------------------------
>
>                 Key: LUCENE-5114
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5114
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Michael McCandless
>            Assignee: Michael McCandless
>             Fix For: 5.0, 4.5
>
>         Attachments: LUCENE-5114.patch
>
>
> Long ago terms dict had a cache, but it was problematic and we removed it, 
> but the API still has a relic boolean useCache ... I think we should drop it 
> from the API as well.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to