allegro / allegro/consul-recipes

BackoffRunner can overflow

Open
#5 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
12
Forks
2
PR merge metrics
No merged PRs in 30d

Description

With big enough `initialBackoff` and `retry`, the `backoff` variable can be negative.

AC:
* `backoff` cannot be be negative. We should set the result of `initialBackoff << retry` to `Integer.MAX_INT` if it's negative.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.