[LUCENE-1330] 0 position increment not properly supported for the first token

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Duplicate
• Component/s: core/store, general/javadocs
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Moti Nisenson
• Created: 08/Jul/08 08:08
• Updated: 28/Aug/22 11:51
• Resolved: 22/Sep/08 17:47

Description

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.