You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jackrabbit.apache.org by "Alex Parvulescu (JIRA)" <ji...@apache.org> on 2011/03/21 15:52:05 UTC

[jira] [Updated] (JCR-2916) Avoid String.intern() when indexing

     [ https://issues.apache.org/jira/browse/JCR-2916?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alex Parvulescu updated JCR-2916:
---------------------------------

    Attachment: JCR-2916-fields.patch

Attaching patch for lucene Field creation. I replaced all the Field constructors with the proper ones to skip the intern() call.

The same issue seems to apply to org.apache.lucene.index.Term, it is also heavily used, and now provides a constructor that allows skipping the intern() call.

Do you think that should be addressed in this issue too?

> Avoid String.intern() when indexing
> -----------------------------------
>
>                 Key: JCR-2916
>                 URL: https://issues.apache.org/jira/browse/JCR-2916
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Marcel Reutegger
>            Priority: Minor
>         Attachments: JCR-2916-fields.patch
>
>
> Lucene 3.0 now allows to create Fields with a String name that is already interned. We should use the new constructor in NodeIndexer to avoid unnecessary String.intern() calls. The field names Jackrabbit uses are available in FieldNames and already interned.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira