Thanks to visit codestin.com
Credit goes to github.com

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

Conversation

@noahdietz
Copy link
Contributor

Disable resolution of the gRPC ServiceConfig from the DNS TXT records to prevent sudden duplication of retry logic should a service owner publish a RetryPolicy. More context in googleapis/gapic-generator#2778.

@noahdietz noahdietz requested a review from andreamlin June 5, 2019 19:45
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 5, 2019
@codecov
Copy link

codecov bot commented Jun 5, 2019

Codecov Report

Merging #731 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #731      +/-   ##
============================================
+ Coverage     77.43%   77.44%   +<.01%     
  Complexity     1081     1081              
============================================
  Files           196      196              
  Lines          4707     4708       +1     
  Branches        369      369              
============================================
+ Hits           3645     3646       +1     
  Misses          895      895              
  Partials        167      167
Impacted Files Coverage Δ Complexity Δ
...api/gax/grpc/InstantiatingGrpcChannelProvider.java 77.97% <100%> (+0.13%) 36 <0> (ø) ⬇️

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 6aaa370...e8be19e. Read the comment docs.

Copy link
Contributor

@andreamlin andreamlin left a comment

Choose a reason for hiding this comment

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

LGTM when tests pass

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.

3 participants