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

Shai Erera commented on LUCENE-2548:
------------------------------------

I agree. Term is frequently used (at least in our apps) and the wrapping around 
BytesRef is nice too. One can still call text() or the like and get the string 
rep. of it which in most cases is what you put there in the first place.

And I also agree about stopping interning field suddenly. What is the reason 
for stop doing that?

> 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]

Reply via email to