Field with an explicit TokenStream must be tokenized and then uses the default Analyzer offset gaps [LUCENE-7267]
- Dominant language
- Java
- Stars
- 3.6k
- Forks
- 1.4k
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 88
Description
This took me somewhat by surprise. We have a pretty complex code that uses fields with explicit token streams (which provide their own offset data) and multivalues.
It was surprising to see that offsets for subsequent values were shifted by 1 compared to what was explicitly provided in the OffsetAttribute. A bit of debugging showed this code inside `PerField.invert`:
```Java
if (analyzed) {
invertState.position += docState.analyzer.getPositionIncrementGap(fieldInfo.name);
invertState.offset += docState.analyzer.getOffsetGap(fieldInfo.name);
}
```
A field with an explicit token stream must still be declared as tokenized and PerField then thinks that this field must have come from an analyzer (where in fact it didn't):
```Java
final boolean analyzed = fieldType.tokenized() && docState.analyzer != null;
```
While the default position increment is 0, the default offset gap isn't – it's 1, causing the shift.
Thoughts?
---
Migrated from [LUCENE-7267](https://issues.apache.org/jira/browse/LUCENE-7267) by Dawid Weiss (@dweiss), updated May 02 2016
Contributor guide
Assessment
This issue has not been assessed yet.