jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Parvulescu (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (JCR-2916) Avoid String.intern() when indexing
Date Mon, 21 Mar 2011 14:52:05 GMT

     [ 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

Mime
View raw message