wanyingd1996

Deprecate the old API, and make the new API delegate to the old API by default. Then users has to at least override one of them. If both are overriden,...

Hi, any thoughts on this proposal? An alternative way with regard to compatibility issue: ``` interface CopyableThreadContextElement { fun copyForChild(): CopyableThreadContextElement fun copyForChildOnNewCoroutine(): CopyableThreadContextElement = copyForChild() } ``` Make `copyForChild`...

Hi, any updates for the issue? having distinction between new coroutine and switching context will make our tracing clients life easier, so that they don't need to manually set ThreadLocal...

I reproduced the bug using InProcessChannel, and the trace shows the problem is inside of `notifyClientClose` method in `InProcessTransport.java`. The `streamClosed` method is called by `clientStream.statsTraceCtx.streamClosed(clientStatus);`, and if this fails,...

@ericgribkoff , do you think we should go with the fix suggested by @groakley ?

@ericgribkoff I think catching the exception in `StatsTraceContext` instead of in `notifyClientClose` would be clear, but it seems I can't shutdown the transport in `StatsTraceContext`. I am fairly new to...

> @wanyingd1996 Are you still interested in working on this issue? Hi, Larry, I'm not working on grpc anymore, and I don't have the bandwidth for the bug now. feel...