Skip to content

Commit

Permalink
archive-client: base retry warning condition on number of retries
Browse files Browse the repository at this point in the history
  • Loading branch information
eldargab committed Sep 11, 2023
1 parent 689c69e commit e849005
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion util/util-internal-archive-client/src/client.ts
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ export class ArchiveClient {
if (this.http.isRetryableError(err)) {
let pause = this.retrySchedule[Math.min(retries, this.retrySchedule.length - 1)]
if (this.log?.isWarn()) {
let warn = pause > 30_000 || err instanceof HttpTimeoutError && err.ms > 10_000
let warn = retries > 3 || err instanceof HttpTimeoutError && err.ms > 10_000
if (warn) {
this.log.warn({
reason: err.message,
Expand Down

0 comments on commit e849005

Please sign in to comment.