Results 96 comments of Kannan J

For the 168 calls with status for the cancelled calls marked as "other" grpc_method, the method descriptor was never obtained as the stream was closed before the call even started....

Looking at the grpc_method="other",grpc_status="CANCELLED" case, it [looks like we don't check for cancellation](https://github.com/grpc/grpc-java/blob/4c7399910246b4111940faf9c3a277b87e559b8a/core/src/main/java/io/grpc/internal/ServerImpl.java#L641) while doing the method lookup and before calling the application. So the ServerTracer is recording "other" because...

I mentioned the cases where the ServerCall.close won't be called is bacause this call originates from the application code and the onCompleted/onError callback on the stream observer being bypassed in...

There may be something here but we are unable to make more headway with the provided information, hence closing the issue. Please comment to reopen if needed.

> `MODULE.bazel` also needs updating. That is why the Bazel test is failing. Done.

@ejona86 Windows test is timing out. Any idea about this [test](https://github.com/grpc/grpc-java/blob/master/servlet/src/tomcatTest/java/io/grpc/servlet/TomcatInteropTest.java)?

Once I noticed there were unintended commits that caused test failure I panic reverted them but I didn't notice there were more commits to revert (there were 4 but I...

Actually the PR was meant to be not just for documentation update but to include the Dockerfiles as well. I realized I had forgotten to do a git add for...

Yes, it uses the calling thread so if you don't call read it might not do the things you expect. What is the pattern you are using here for the...

I hope your issue is resolved. Feel free to comment to reopen in the future if needed.