[
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: [email protected]
For additional commands, e-mail: [email protected]
