[ 
https://issues.apache.org/jira/browse/LUCENE-1330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12615191#action_12615191
 ] 

Michael McCandless commented on LUCENE-1330:
--------------------------------------------

In LUCENE-1255, we tried to correct DocumentsWriter to write absolute position 
as 0 not -1 in this case, but unfortunately this broke backwards compatibility 
so we decided to leave it be.

But: in this case the absolute position should read back later as -1, not 
Integer.MIN_VALUE -- where are you seeing Integer.MIN_VALUE?

> 0 position increment not properly supported for the first token
> ---------------------------------------------------------------
>
>                 Key: LUCENE-1330
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1330
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Javadocs, Store
>            Reporter: Moti Nisenson
>
> Setting a position increment of 0 for the first token in a field results in 
> its "absolute position" (as well as increment) being read back later as 
> Integer.MIN_VALUE.
> This is a result of how the information gets written out in DocumentsWriter: 
> position should not be updated using += t.getPositionIncrement() - 1; and 
> then always ++'ed in addPosition. It would be much simpler just to update it 
> using t.getPositionIncrement().
> While this is fairly easy to fix in DocumentsWriter, one could just update 
> the documentation for Token.setPositionIncrement() to indicate that for 
> indexing purposes the first Token in a field must have a positive position 
> increment.

-- 
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to