-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
Fix automatic tracking of collapse with docvalue_fields #110103
Fix automatic tracking of collapse with docvalue_fields #110103
Conversation
Pinging @elastic/es-search (Team:Search) |
Hi @benwtrent, I've created a changelog YAML for you. |
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
summary: Fix automatic tracking of collapse with `docvalue_fields` | ||
area: Search | ||
type: bug | ||
issues: [] |
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.
Nit: Add #96510 to issue list
@elasticmachine update branch |
Hi @benwtrent, I've updated the changelog YAML for you. |
💚 Backport successful
|
There were some optimizations that broke collapse fields automatically being added to `docvalue_fields` during the fetch phase. Consequently, users will get really weird errors like `unsupported_operation_exception`. This commit corrects the intended behavior of automatically including the collapse field in the docvalue_fields context during fetch if it isn't already included. closes: elastic#96510
…10154) There were some optimizations that broke collapse fields automatically being added to `docvalue_fields` during the fetch phase. Consequently, users will get really weird errors like `unsupported_operation_exception`. This commit corrects the intended behavior of automatically including the collapse field in the docvalue_fields context during fetch if it isn't already included. closes: #96510
There were some optimizations that broke collapse fields automatically being added to
docvalue_fields
during the fetch phase.Consequently, users will get really weird errors like
unsupported_operation_exception
. This commit corrects the intended behavior of automatically including the collapse field in the docvalue_fields context during fetch if it isn't already included.closes: #96510