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

Add retry to ESIndex class methods #2171

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

artem-shelkovnikov
Copy link
Member

@artem-shelkovnikov artem-shelkovnikov commented Feb 15, 2024

Part of https://github.com/elastic/enterprise-search-team/issues/6412

This PR adds same retry mechanism to ESIndex class - it got overlooked

Checklists

Pre-Review Checklist

  • this PR has a meaningful title
  • this PR links to all relevant github issues that it fixes or partially addresses
  • if there is no GH issue, please create it. Each PR should have a link to an issue
  • this PR has a thorough description
  • Covered the changes with automated tests
  • Tested the changes locally
  • Added a label for each target release version (example: v7.13.2, v7.14.0, v8.0.0)

@artem-shelkovnikov artem-shelkovnikov merged commit 0c5b735 into main Feb 15, 2024
3 checks passed
@artem-shelkovnikov artem-shelkovnikov deleted the artem/add-retry-to-es-index branch February 15, 2024 20:24
github-actions bot pushed a commit that referenced this pull request Feb 15, 2024
Copy link

💚 Backport PR(s) successfully created

Status Branch Result
8.13 #2176

This backport PR will be merged automatically after passing CI.

artem-shelkovnikov added a commit that referenced this pull request Feb 15, 2024
Co-authored-by: Artem Shelkovnikov <artem.shelkovnikov@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants