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

[Fix #812] Add the ignore_unavailable option to the request #819

Merged
merged 1 commit into from
Oct 28, 2021

Conversation

kolauren
Copy link
Contributor

@kolauren kolauren commented Oct 28, 2021

Add the ignore_unavailable option to the request: https://www.elastic.co/guide/en/elasticsearch/client/curator/current/option_ignore.html

This solves #812


Before submitting the PR make sure the following are checked:

  • The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
  • Wrote good commit messages.
  • Commit message starts with [Fix #issue-number] (if the related issue exists).
  • Feature branch is up-to-date with master (if not - rebase it).
  • Squashed related commits together.
  • Added tests.
  • Added an entry to the changelog if the new code introduces user-observable changes. See changelog entry format for details.

kolauren pushed a commit to kolauren/chewy that referenced this pull request Oct 28, 2021
@kolauren kolauren marked this pull request as ready for review October 28, 2021 00:11
@rabotyaga
Copy link
Contributor

Hey @kolauren 👋 !
Thanks for the contribution!
Could you please change all links from curator reference to Elastcisearch reference: https://www.elastic.co/guide/en/elasticsearch/reference/current/multi-index.html#multi-index ?

@kolauren
Copy link
Contributor Author

Updated the links!

@rabotyaga
Copy link
Contributor

@kolauren Thank you very much! 🙇‍♂️ 🙏

@rabotyaga rabotyaga merged commit d0aef50 into toptal:master Oct 28, 2021
cyucelen pushed a commit to cyucelen/chewy that referenced this pull request Jan 28, 2023
cyucelen pushed a commit to cyucelen/chewy that referenced this pull request Jan 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants