Skip to content
This repository was archived by the owner on Sep 26, 2023. It is now read-only.

Conversation

@igorbernstein2
Copy link
Contributor

@igorbernstein2 igorbernstein2 commented Oct 10, 2018

When RPC timeouts are not configured in RetrySettings, but a caller passes an RPC timeout, we should respect it.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 10, 2018
@codecov-io
Copy link

codecov-io commented Oct 10, 2018

Codecov Report

Merging #602 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #602      +/-   ##
============================================
+ Coverage     75.01%   75.02%   +0.01%     
- Complexity      933      934       +1     
============================================
  Files           176      176              
  Lines          4074     4076       +2     
  Branches        322      323       +1     
============================================
+ Hits           3056     3058       +2     
  Misses          865      865              
  Partials        153      153
Impacted Files Coverage Δ Complexity Δ
...n/java/com/google/api/gax/rpc/AttemptCallable.java 76.19% <100%> (+2.5%) 4 <0> (+1) ⬆️

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 12d0c5f...3889722. Read the comment docs.

Copy link
Contributor

@vam-google vam-google left a comment

Choose a reason for hiding this comment

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

LGTM

@igorbernstein2 igorbernstein2 merged commit b842b23 into googleapis:master Oct 10, 2018
@igorbernstein2 igorbernstein2 deleted the opencensus-1c-fix-bug branch October 10, 2018 20:17
Raibaz pushed a commit to Raibaz/gax-java that referenced this pull request Oct 25, 2018
Raibaz pushed a commit to Raibaz/gax-java that referenced this pull request Oct 25, 2018
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.

4 participants