Stack Overflow when running AsyncClient

[carl-mastrangelo]

INFO: [ManagedChannelImpl@506e6d5e] Created with target directaddress:///localhost/127.0.0.1:1234
May 27, 2016 9:43:05 AM io.grpc.internal.ManagedChannelImpl <init>
INFO: [ManagedChannelImpl@668bc3d5] Created with target directaddress:///localhost/127.0.0.1:1234
May 27, 2016 9:43:05 AM io.grpc.internal.ManagedChannelImpl <init>
INFO: [ManagedChannelImpl@62043840] Created with target directaddress:///localhost/127.0.0.1:1234
Exception in thread "grpc-default-executor-32" java.lang.StackOverflowError
        at io.grpc.Context.removeListener(Context.java:454)
        at io.grpc.Context.removeListener(Context.java:462)
        at io.grpc.Context.removeListener(Context.java:462)
        at io.grpc.Context.removeListener(Context.java:462)
        at io.grpc.Context.removeListener(Context.java:462)
        at io.grpc.Context.removeListener(Context.java:462)
        at io.grpc.Context.removeListener(Context.java:462)
        at io.grpc.Context.removeListener(Context.java:462)
        at io.grpc.Context.removeListener(Context.java:462)

[carl-mastrangelo]

I suspect that b5e6d42 is somehow messing up the parents to be recursive.

[ejona86]

It seems to me like this would have been an issue ever since Context was added to client-side:
https://github.com/grpc/grpc-java/blob/1cc76d8/core/src/main/java/io/grpc/internal/ClientCallImpl.java#L167
https://github.com/grpc/grpc-java/blob/a7049bc/benchmarks/src/main/java/io/grpc/benchmarks/qps/AsyncClient.java#L202

[carl-mastrangelo]

@ejona86 Yup, I think you're right. Not sure what the fix is, I ran out of time on Friday looking into this.

[ejona86]

@carl-mastrangelo, do you want to take this issue? It should be as easy as setting the context to ROOT before each RPC.

[carl-mastrangelo]

Yeah, I have a PR, I just hadn't sent it out.