-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Search service] Rename async search param #61641
Conversation
Pinging @elastic/kibana-app-arch (Team:AppArch) |
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.
The rename is good, but did you intend to commit the Jeninsfile
too?
@elasticmachine merge upstream |
e655d0a
to
33522c6
Compare
@elasticsearch merge upstream |
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
* Rename async search params (#61641) Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> * Remove polling delay * Fix merge issue Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Liza Katz <lizka.k@gmail.com>
Summary
Summarize your PR. If it involves visual changes include a screenshot or gif.
Fixes #61498.
The
_async_search
parameterwait_for_completion
was renamed towait_for_completion_timeout
in elastic/elasticsearch#54198. This PR updates the name of the parameter in our search service.