Skip to content
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 query param to limit highlighting to specified length (#67325) #69016

Merged
merged 1 commit into from
Feb 16, 2021

Conversation

matriv
Copy link
Contributor

@matriv matriv commented Feb 16, 2021

Add a max_analyzed_offset query parameter to allow users
to limit the highlighting of text fields to a value less than or equal to the
index.highlight.max_analyzed_offset, thus avoiding an exception when
the length of the text field exceeds the limit. The highlighting still takes place,
but stops at the length defined by the new parameter.

Closes: #52155
(cherry picked from commit f9af60b)

)

Add a `max_analyzed_offset` query parameter to allow users
to limit the highlighting of text fields to a value less than or equal to the
`index.highlight.max_analyzed_offset`, thus avoiding an exception when
the length of the text field exceeds the limit. The highlighting still takes place,
but stops at the length defined by the new parameter.

Closes: elastic#52155
(cherry picked from commit f9af60b)
@matriv matriv added :Search Relevance/Highlighting How a query matched a document backport labels Feb 16, 2021
@elasticmachine elasticmachine added the Team:Search Meta label for search team label Feb 16, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search (Team:Search)

@matriv matriv merged commit 931e149 into elastic:7.x Feb 16, 2021
@matriv matriv deleted the backport-67325-7.x branch February 16, 2021 11:32
matriv added a commit to matriv/elasticsearch that referenced this pull request Feb 16, 2021
Since elastic#69016 is merged versions for serialisation/deserialisation
and YAML tests are updated.

Follows: elastic#67325
matriv added a commit that referenced this pull request Feb 16, 2021
Since #69016 is merged versions for serialisation/deserialisation
and YAML tests are updated.

Follows: #67325
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :Search Relevance/Highlighting How a query matched a document Team:Search Meta label for search team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants