waziqi89 opened a new pull request, #12222: URL: https://github.com/apache/lucene/pull/12222
### Description This is a fix to https://github.com/apache/lucene/issues/12221 . FragEnd was the start of the next potential fragment, which leading the boundary scanner to cover the first word in the next field when the fragment ends at current field's end. Changed a text in `TestSimpleFragmentsBuilder.java`. The behavior change is expected (could be another bug): 1. the newly created fragments [will stop at the original (when discreteMultiValueHighlight is unset) fragment's end](https://github.com/waziqi89/lucene/blob/0c2f9a97af85c3839e0fe62fda90ba76925f8c31/lucene/highlighter/src/java/org/apache/lucene/search/vectorhighlight/BaseFragmentsBuilder.java#L313) even if the newly created fragment can be extended further as the new could fragment start later than the original one. 2. FragEnd used to be at "t" of the next word "text", so boundary scanner will cover the whole word. Now FragEnd stops before that character, so the "text" is excluded entirely -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
