Skip to content

add config request retrying#20

Merged
ajwootto merged 1 commit intomainfrom
DVC-1841-request-retries
Jan 6, 2022
Merged

add config request retrying#20
ajwootto merged 1 commit intomainfrom
DVC-1841-request-retries

Conversation

@ajwootto
Copy link
Copy Markdown
Contributor

@ajwootto ajwootto commented Jan 5, 2022

No description provided.

@ajwootto ajwootto force-pushed the DVC-1841-request-retries branch from 205dbbe to 9b91958 Compare January 6, 2022 15:30
@ajwootto ajwootto marked this pull request as ready for review January 6, 2022 15:32
block: suspend () -> T): T
{
var currentDelay = initialDelay
repeat(times - 1) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this basically a loop forever?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's a loop until the max delay is reached unless you pass in a smaller number for "times", in which case it'll end after that number of times even if the delay hasn't been reached

@ajwootto ajwootto force-pushed the DVC-1841-request-retries branch 2 times, most recently from b618431 to 4eda5f8 Compare January 6, 2022 20:54
@ajwootto ajwootto force-pushed the DVC-1841-request-retries branch from 4eda5f8 to c193cc4 Compare January 6, 2022 20:57
@ajwootto ajwootto merged commit baef2ac into main Jan 6, 2022
@ajwootto ajwootto deleted the DVC-1841-request-retries branch January 6, 2022 21:32
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