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.

Sign up for GitHub

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

Jump to bottom

Retry.DelayingRetryPolicy does not work with delays #2937

Closed
martin-sladecek opened this issue Apr 15, 2021 · 0 comments · Fixed by #2938
Closed

Retry.DelayingRetryPolicy does not work with delays #2937

martin-sladecek opened this issue Apr 15, 2021 · 0 comments · Fixed by #2938
Assignees
@spericas @martin-sladecek
Labels
bug Something isn't working fault-tolerance P2

Comments

@martin-sladecek
Copy link
Contributor

Environment Details

  • Helidon Version: 2.2.2
  • Helidon SE or Helidon MP
  • JDK version: 11
  • OS: Linux
  • Docker version (if applicable):

Problem Description

Retry.DelayingRetryPolicy is always repeating without a delay.

Steps to reproduce

Create a DelayingRetryPolicy with delay(Duration.ofSeconds(5)) and delayFactor(1.0) - it won't do any delays.
martin-sladecek added a commit to martin-sladecek/helidon that referenced this issue Apr 15, 2021
@martin-sladecek
Fix delay computation in Retry policies
f60b873 
Resolves helidon-io#2937
@martin-sladecek martin-sladecek mentioned this issue Apr 15, 2021
Merged
@barchetta barchetta added P2 bug Something isn't working labels Apr 15, 2021
spericas pushed a commit that referenced this issue Apr 19, 2021
@martin-sladecek
Fix delay computation in Retry policies (#2938)
aac3eba 
* Fix delay computation in Retry policies

Resolves #2937

* 2937 Remove PartialPublisher (dead code)
aseovic pushed a commit to aseovic/helidon that referenced this issue Apr 26, 2021
@martin-sladecek @aseovic
Fix delay computation in Retry policies (helidon-io#2938)
f27f441 
* Fix delay computation in Retry policies

Resolves helidon-io#2937

* 2937 Remove PartialPublisher (dead code)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spericas spericas

@martin-sladecek martin-sladecek

Labels
bug Something isn't working fault-tolerance P2
Projects
Backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix delay computation in Retry policies martin-sladecek/helidon
3 participants
@spericas @barchetta @martin-sladecek