David Smiley created LUCENE-6898:
------------------------------------

             Summary: Avoid reading last stored field value when 
StoredFieldVisitor.Status.NO
                 Key: LUCENE-6898
                 URL: https://issues.apache.org/jira/browse/LUCENE-6898
             Project: Lucene - Core
          Issue Type: Improvement
          Components: core/codecs
            Reporter: David Smiley
            Assignee: David Smiley
            Priority: Minor


CompressingStoredFieldsReader.visitDocument (line 597) loops through the fields 
in the input while consulting the StoredFieldVisitor on what to do.  There is a 
small optimization that could be done on the last loop iteration.  If the 
visitor returns Status.NO then it should be treated as equivalent to 
Status.STOP.  As it is now, it will call skipField() which reads needless bytes 
from the DataInput that won't be used.

With this optimization in place, it is advisable to put the largest text field 
last in sequence -- something the user or search platform (e.g. ES/Solr) could 
do.



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