[ https://issues.apache.org/jira/browse/LUCENE-1255?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12592898#action_12592898 ]
Michael Busch commented on LUCENE-1255: --------------------------------------- I don't think this was a backwards-compatible change. With this change you can't tell anymore if the first token had a positionIncrement set to 0 or 1. TermPositions.nextPosition() returns 0 in both cases. Applications today might rely on getting -1 from nextPosition() in the former case. > CheckIndex should allow term position = -1 > ------------------------------------------ > > Key: LUCENE-1255 > URL: https://issues.apache.org/jira/browse/LUCENE-1255 > Project: Lucene - Java > Issue Type: Bug > Components: Index > Affects Versions: 2.4 > Reporter: Michael McCandless > Assignee: Michael McCandless > Priority: Minor > Fix For: 2.3.2, 2.4 > > Attachments: LUCENE-1255.patch, LUCENE-1255.take2.patch > > > Spinoff from this discussion: > > http://mail-archives.apache.org/mod_mbox/lucene-java-user/200803.mbox/[EMAIL > PROTECTED] > Right now CheckIndex claims the index is corrupt if you index a Token with -1 > position, which happens if your first token has positionIncrementGap set to 0. > But, as far as I can tell, Lucene doesn't "mind" when this happens. > So I plan to fix CheckIndex to allow this case. I'll backport to 2.3.2 as > well. > LUCENE-1253 is one example where Lucene's core analyzers could do this. -- 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]