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

[BUG] Request context cancellation is ignored when retryBackoff is configured #539

Closed
zhyu opened this issue May 7, 2024 · 0 comments · Fixed by #540
Closed

[BUG] Request context cancellation is ignored when retryBackoff is configured #539

zhyu opened this issue May 7, 2024 · 0 comments · Fixed by #540
Labels
bug Something isn't working

Comments

@zhyu
Copy link
Contributor

zhyu commented May 7, 2024

When retryBackoff is configured, the client sleeps between retries and ignores request context cancellation.

In short, the same issue as the one in the origin ES client. The fix has not yet been ported to the OpenSearch client

@zhyu zhyu added bug Something isn't working untriaged labels May 7, 2024
zhyu added a commit to zhyu/opensearch-go that referenced this issue May 8, 2024
@dblock dblock removed the untriaged label May 16, 2024
Jakob3xD pushed a commit to zhyu/opensearch-go that referenced this issue Jul 1, 2024
Jakob3xD added a commit that referenced this issue Jul 1, 2024
…540)

* fix request context cancellation is ignored when retryBackoff (#539)

Signed-off-by: zhyu <angellwings@gmail.com>

* fix unexpected sleep after the last retry

Signed-off-by: zhyu <angellwings@gmail.com>

* add guides on configuring the client with retry and backoff

Signed-off-by: zhyu <angellwings@gmail.com>

* Update guides/retry_backoff.md

Co-authored-by: Jakob <jakob.hahn@hetzner.com>
Signed-off-by: Yu Zhang <angellwings@gmail.com>

---------

Signed-off-by: zhyu <angellwings@gmail.com>
Signed-off-by: Yu Zhang <angellwings@gmail.com>
Co-authored-by: Jakob <jakob.hahn@hetzner.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants