Doug Fawley
Doug Fawley
> remove the test case https://github.com/grpc/grpc-go/blob/master/clientconn_test.go#L390 as we are not going to support timeout feature. As I've said several times now, we _are_ going to support the timeout feature. We...
> then, please, shall we make this test case without deprecated functions as Dial, WithTimeout, WithBlock? I don't know how to do what you're requesting. The only options are: 1....
From chat offline: > Actually, this structure of the test actually proves there is a bug here that I don't think you're fixing > If a connection is created, we...
Looks like you still need to exclude these tests from go1.20 runs, too.
> write another function that allows the arguments to be processed lazily (i.e. pass in closures). Upon further consideration, I don't think this was a good suggestion.
Yes, that's what I was thinking, too. We could add this method -- worst case, the `PrefixLogger` can expose it and just call the global logger like it's doing today.
You're welcome to attempt it, but it could be tricky. Something from this list might be a little more new-to-grpc friendly: https://github.com/grpc/grpc-go/issues?q=is%3Aopen+label%3A%22Status%3A+Help+Wanted%22+sort%3Aupdated-desc
Unfortunately we don't have the hardware needed to debug this. If someone else can help, we can review and merge PRs. Thanks!
Something like this could be really interesting. Unfortunately this is a very niche kind of thing and we likely won't have the bandwidth to work on it in the foreseeable...
I definitely would like to put the interfaces in `experimental` first, to prevent anyone from relying upon them until all of the work is done and we're sure they are...