[
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
