Conversation
Codecov Report
@@ Coverage Diff @@
## master #1160 +/- ##
============================================
- Coverage 79.07% 78.97% -0.11%
- Complexity 1193 1194 +1
============================================
Files 205 205
Lines 5258 5266 +8
Branches 433 436 +3
============================================
+ Hits 4158 4159 +1
- Misses 929 935 +6
- Partials 171 172 +1
Continue to review full report at Codecov.
|
gax/src/main/java/com/google/api/gax/retrying/BasicRetryingFuture.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont think this is a good idea.
Logging should be reserved for unexpected errors and for things that callers can do something about. Retries should be considered part of normal operation. I think this will increase support load for service teams unnecessarily.
Can we discuss this offline before merging this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (with a small change request)
gax/src/main/java/com/google/api/gax/retrying/BasicRetryingFuture.java
Outdated
Show resolved
Hide resolved
Spoke offline and this seems like the best option in the short term future
Fixes b/160995457