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

Uwe Schindler edited comment on LUCENE-2545 at 7/18/10 4:26 AM:
----------------------------------------------------------------

For trunk the usage of intern() may be removed completely from Lucene, as 
fields are now decoupled from terms (its a separate thing in index).

Some pachanges in the patch may help for 3.x branch (not 3.0!), but we should 
not expose APIs to the outside that accept a "intern" boolean parameter (e.g. 
Field, AbstractField, NumericField).

      was (Author: thetaphi):
    For trunk the usage of intern() may be removed completely from Lucene, as 
fields are now decoupled from terms (its a separate thing in index).

Some pachanges in the patch may help for 3.x branch (not 3.0!), but we should 
not expose APIs to the outside that accept a "intern" boolean parameter (e.g. 
NumericField).
  
> improve uses of StringHelper.intern in Field,AbstractField, NumericField etc.
> -----------------------------------------------------------------------------
>
>                 Key: LUCENE-2545
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2545
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Other
>    Affects Versions: 3.0.2
>            Reporter: Woody Anderson
>             Fix For: 3.1, 4.0
>
>         Attachments: LUCENE-2545.patch
>
>
> There are many times for certain field types that intern is still called when 
> it is not needed. We can improve this with pretty simple open up of the 
> constructors etc.

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