-
Notifications
You must be signed in to change notification settings - Fork 568
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
Fix RealGrpcCall timeout #2816
Fix RealGrpcCall timeout #2816
Conversation
72c462b
to
62e9398
Compare
@@ -85,6 +86,9 @@ class GrpcClientTest { | |||
private lateinit var incompatibleRouteGuideService: IncompatibleRouteGuideClient | |||
private var callReference = AtomicReference<Call>() | |||
|
|||
// Just set to a large enough value we won't timeout in tests | |||
private val okHttpClientTimeout = Duration.ofSeconds(117) |
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.
This value is SO BANANAS
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.
🍌
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.
Thank you
@@ -85,6 +86,9 @@ class GrpcClientTest { | |||
private lateinit var incompatibleRouteGuideService: IncompatibleRouteGuideClient | |||
private var callReference = AtomicReference<Call>() | |||
|
|||
// Just set to a large enough value we won't timeout in tests | |||
private val okHttpClientTimeout = Duration.ofSeconds(117) |
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.
🍌
LateInitTimeout
doesn't actually work properly, and overwrites the new with the old, instead of the old with the new. I'm not convinced it's needed, and since it's internal to Wire, I'm hoping I can just delete it and useForwardingTimeout
instead.Added a test to verify that the timeout is preserved after
initCall
is called.