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

Robert Muir updated LUCENE-6325:
--------------------------------
    Attachment: LUCENE-6325.patch

Here was my first patch (moved from LUCENE-6320).

Maybe we can add some fieldinfos tests to exercise this directly, and maybe 
there is a way to simplify the scary logic too.

> improve perf and memory of FieldInfos.fieldInfo(int)
> ----------------------------------------------------
>
>                 Key: LUCENE-6325
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6325
>             Project: Lucene - Core
>          Issue Type: Sub-task
>            Reporter: Robert Muir
>             Fix For: Trunk, 5.1
>
>         Attachments: LUCENE-6325.patch
>
>
> FieldInfos.fieldInfo(int) looks up a field by number and returns its 
> FieldInfo.
> This method is called per-field-per-doc in things like stored fields and 
> vectors readers.
> Unfortunately, today this method is always backed by a TreeMap. In most cases 
> a simple array is better, its faster and uses less memory.
> These changes made significant difference in stored fields checkindex time 
> with my test index (had only 10 fields). Maybe it helps merge as well.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to