-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
Remove parent query string parameter #41098
Remove parent query string parameter #41098
Conversation
This commit removes the deprecated parent query string parameter. The routing parameter should be used instead.
Pinging @elastic/es-distributed |
Pinging @elastic/es-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.
Thanks @russcam , should we also merge in 7x ?
This commit removes the generated obsolete parameters through code generation. Relates: elastic/elasticsearch#41098
@elasticmachine run elasticsearch-ci/packaging-sample |
This commit removes the deprecated parent query string parameter. The routing parameter should be used instead.
Thanks @russcam , I merged in master and 7.x |
Hey @jimczi many thanks fo pulling this in. For 7.x I don't think we can do a hard remove for similar reasons as outlined here: |
@jimczi Please remember to add all relevant labels (area label, version label(s) and change type label) on all PRs and please look for this as part of reviews. The release note generation process is made much harder when PRs are not labelled correctly. |
This commit removes the deprecated parent query string parameter. The routing parameter should be used instead.
This commit removes the deprecated
parent
query string parameter. The routing parameter should be used instead.