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

Robert Muir commented on LUCENE-8192:
-------------------------------------

Second, less aggressive patch: it *changes the boolean* from 
{{offsetsAreCorrect}} to {{graphOffsetsAreCorrect}} and always enables the 
checks consistent with what indexwriter will do (e.g. offsets don't go 
backwards).

I ran tests a couple times and nothing failed... probably deserves some 
beasting but I think this is a good step? It removes some useless leniency.

> Remove offsetsAreCorrect from BaseTokenStreamTestCase
> -----------------------------------------------------
>
>                 Key: LUCENE-8192
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8192
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>            Priority: Major
>         Attachments: LUCENE-8192_prototype.patch, LUCENE-8192_take_two.patch
>
>
> Similar to LUCENE-8191, now that indexwriter checks the offsets, this boolean 
> is useless: if offsets are broken it will still fail.
> We should just remove the boolean.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to