-
Notifications
You must be signed in to change notification settings - Fork 24.6k
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
Propagate original indices in NodeTermsEnumRequest #77776
Conversation
This fix ensures that we provide the original list of indices as part of the node-level terms enum request. Closes elastic#77508
Pinging @elastic/es-search (Team:Search) |
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 - the only question I have is whether the rest tests cover the expected behaviour for the various combinations of security on aliases and underlying indices. E.g.
- Security granted to alias with no conditions on underlying indices
- Security granted to alias with mixed conditions also defined on underlying indices
I think we agreed it was OK for terms_enum to ignore any filters associated with aliases.
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
This fix ensures that we provide the original list of indices as part of the node-level terms enum request. Closes elastic#77508
This fix ensures that we provide the original list of indices
as part of the node-level terms enum request.
Closes #77508