Skip to content
This repository was archived by the owner on Nov 18, 2025. It is now read-only.

fix: fix possible invalid retry variables#682

Merged
xiaozhenliu-gg5 merged 2 commits intomasterfrom
retry
Dec 13, 2019
Merged

fix: fix possible invalid retry variables#682
xiaozhenliu-gg5 merged 2 commits intomasterfrom
retry

Conversation

@xiaozhenliu-gg5
Copy link
Contributor

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 13, 2019
@codecov
Copy link

codecov bot commented Dec 13, 2019

Codecov Report

Merging #682 into master will decrease coverage by 0.02%.
The diff coverage is 60%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     googleapis/gax-nodejs#682      +/-   ##
==========================================
- Coverage   92.09%   92.06%   -0.03%     
==========================================
  Files          39       39              
  Lines        7324     7324              
  Branches      404      407       +3     
==========================================
- Hits         6745     6743       -2     
- Misses        576      578       +2     
  Partials        3        3
Impacted Files Coverage Δ
src/normalCalls/retries.ts 94.15% <60%> (-1.17%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6ecdeb7...4c8e40f. Read the comment docs.

@alexander-fenster
Copy link
Contributor

I have no idea how this code works, let's talk tomorrow :)

@alexander-fenster
Copy link
Contributor

Okay now I see how it works, thank you!

@xiaozhenliu-gg5 xiaozhenliu-gg5 merged commit 2cb343c into master Dec 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Exponential backoff retry strategy

3 participants