-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Add positions index_option to patterned_text #133679
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add positions index_option to patterned_text #133679
Conversation
Pinging @elastic/es-storage-engine (Team:StorageEngine) |
x-pack/plugin/logsdb/src/yamlRestTest/resources/rest-api-spec/test/patternedtext/10_basic.yml
Outdated
Show resolved
Hide resolved
...sdb/src/main/java/org/elasticsearch/xpack/logsdb/patternedtext/PatternedTextFieldMapper.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, modulo the default mode.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍 , Let's add a variant of the patterned text benchmark with positions as well?
Add an index_options field to patterned_text type. Currently, only supports docs and positions. When docs is enabled, positions are not stored, and queries fall back to a source confirmed query, like match_only_text. With positions enabled, phrase queries are efficient as the source does not need to be checked. Defaults to docs. Fixes elastic#128935
Add an
index_options
field topatterned_text
type. Currently, only supportsdocs
andpositions
. Whendocs
is enabled, positions are not stored, and queries fall back to a source confirmed query, like match_only_text. Withpositions
enabled, phrase queries are efficient as the source does not need to be checked. Defaults todocs
.Fixes #128935