-
Notifications
You must be signed in to change notification settings - Fork 231
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add interceptor to inject remote address to non-ok status discription #441
Conversation
/gcbrun |
Codecov Report
@@ Coverage Diff @@
## master #441 +/- ##
============================================
- Coverage 81.54% 81.45% -0.10%
Complexity 2016 2016
============================================
Files 134 134
Lines 8877 8890 +13
Branches 1043 1043
============================================
+ Hits 7239 7241 +2
- Misses 1206 1218 +12
+ Partials 432 431 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
gcsio/src/main/java/com/google/cloud/hadoop/gcsio/GoogleCloudStorageGrpcWriteChannel.java
Show resolved
Hide resolved
gcsio/src/main/java/com/google/cloud/hadoop/gcsio/StorageStubProvider.java
Show resolved
Hide resolved
Thanks! This would be helpful to nail the backend! |
…cription (GoogleCloudDataproc#441)" This reverts commit 815e773.
GoogleCloudDataproc#471) * Revert "Add interceptor to inject remote address to non-ok status discription (GoogleCloudDataproc#441)" This reverts commit 815e773.
For debugging purposes, when client receives non-ok status, we want to know the remote peer address.
To achieve this, we need add additional client interceptor to introspect the remote address before error got thrown out, and also inject the address string into the status description.
+cc @veblush