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

[Rest Api Compatibility] Nested path and filter sort options #76022

Merged
merged 5 commits into from
Aug 5, 2021

Conversation

pgomulka
Copy link
Contributor

@pgomulka pgomulka commented Aug 3, 2021

previously removed in #42809. nested_path and nested_filter sort options will now throw exceptions suggesting to use nested option
relates #51816

@pgomulka pgomulka self-assigned this Aug 3, 2021
@pgomulka pgomulka added :Search/Search Search-related issues that do not fall into other categories and removed WIP labels Aug 4, 2021
@pgomulka pgomulka marked this pull request as ready for review August 4, 2021 09:53
@elasticmachine elasticmachine added the Team:Search Meta label for search team label Aug 4, 2021
@elasticmachine
Copy link
Collaborator

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

Copy link
Contributor

@romseygeek romseygeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pgomulka pgomulka merged commit d85acf4 into elastic:master Aug 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team v8.0.0-alpha2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants