[
https://issues.apache.org/jira/browse/LUCENE-2548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12889612#action_12889612
]
Robert Muir commented on LUCENE-2548:
-------------------------------------
Uwe, but removing intern() from Term is almost just as bad as removing Term,
because we at least have to review all uses (e.g. Solr) and see if it would
cause incorrect code (e.g. == comparison that is suddenly wrong) or performance
problems in containers sorting terms or anything of the like?
Again, I don't personally have an opinion either way, I just mentioned why I
didn't remove it, its like Token, still lots of code using it :)
> Remove all interning of field names from flex API
> -------------------------------------------------
>
> Key: LUCENE-2548
> URL: https://issues.apache.org/jira/browse/LUCENE-2548
> Project: Lucene - Java
> Issue Type: Improvement
> Reporter: Uwe Schindler
> Fix For: 4.0
>
>
> In previous versions of Lucene, interning of fields was important to minimize
> string comparison cost when iterating TermEnums, to detect changes in field
> name. As we separated field names from terms in flex, no query compares field
> names anymore, so the whole performance problematic interning can be removed.
> I will start with doing this, but we need to carefully review some places
> e.g. in preflex codec.
> Maybe before this issue we should remove the Term class completely. :-)
> Robert?
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
