Doug Fawley
Doug Fawley
This is a notice that grpc-go intends to change in a way that we know will break the way cmux currently works by default. This will bring grpc-go in line...
Data race detected: https://github.com/grpc/grpc-go/pull/4549/checks?check_run_id=2844066083#step:8:175
https://github.com/grpc/grpc-go/runs/5744477531?check_suite_focus=true ``` ================== WARNING: DATA RACE Read at 0x00c0001ae930 by goroutine 160: runtime.mapaccess1_fast32() /opt/hostedtoolcache/go/1.18.0/x64/src/runtime/map_fast32.go:13 +0x0 google.golang.org/grpc/internal/transport.testFlowControlAccountCheck() /home/runner/work/grpc-go/grpc-go/internal/transport/transport_test.go:1571 +0xc06 google.golang.org/grpc/internal/transport.s.TestAccountCheckWindowSizeWithSmallWindow() /home/runner/work/grpc-go/grpc-go/internal/transport/transport_test.go:1478 +0x55 google.golang.org/grpc/internal/transport.(*s).TestAccountCheckWindowSizeWithSmallWindow() :1 +0x1d runtime.call16() /opt/hostedtoolcache/go/1.18.0/x64/src/runtime/asm_amd64.s:701 +0x48 reflect.callMethod() :1 +0x64...
@markdroth @ejona86 We actually need to define the requirements before beginning work here. Are these requirements? 1. Start picking at the top of the list for the first RPC. -...
One notable problem here is that we can currently only install one stats handler at a time. This means we can't enable our default tracing and also let the user...
https://github.com/grpc/grpc-go/blob/master/Documentation/server-reflection-tutorial.md#enable-server-reflection Does not specify the `--channel_creds_type=local` flag is needed (EDIT: it isn't needed in OSS, since the default is local credentials). Also consider updating to use `grpcurl` instead of `grpc_cli`.
This would require a codec API change/extension to recycle the memory once grpc is done writing it to the wire (or compressing it).
Also we need a `protoiface.MessageV1` that is allowed to be referenced by outside packages. Background: we have [an API](https://pkg.go.dev/google.golang.org/[email protected]/internal/status#Status.WithDetails) that accepts v1 `proto.Message`s as a parameter. We would like to...
gRPC users would like to distinguish between connection errors and authorization/other errors so they can programmatically retry RPCs that are more likely to succeed on subsequent attempts. It doesn't seem...