lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Smiley (JIRA)" <j...@apache.org>
Subject [jira] Created: (LUCENE-2529) always apply position increment gap between values
Date Tue, 06 Jul 2010 16:30:51 GMT
always apply position increment gap between values
--------------------------------------------------

                 Key: LUCENE-2529
                 URL: https://issues.apache.org/jira/browse/LUCENE-2529
             Project: Lucene - Java
          Issue Type: Improvement
         Environment: (I don't know which version to say this affects since it's some quasi
trunk release and the new versioning scheme confuses me.)
            Reporter: David Smiley


I'm doing some fancy stuff with span queries that is very sensitive to term positions.  I
discovered that the position increment gap on indexing is only applied between values when
there are existing terms indexed for the document.  I suspect this logic wasn't deliberate,
it's just how its always been for no particular reason.  I think it should always apply the
gap between fields.  Reference DocInverterPerField.java line 82:

if (fieldState.length > 0)
          fieldState.position += docState.analyzer.getPositionIncrementGap(fieldInfo.name);

This is checking fieldState.length.  I think the condition should simply be:  if (i > 0).
I don't think this change will affect anyone at all but it will certainly help me.  Presently,
I can either change this line in Lucene, or I can put in a hack so that the first value for
the document is some dummy value which is wasteful.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message