Lucio Franco

Results 524 comments of Lucio Franco

``` TimeoutOnSleepingServer: x code must be DeadlineExceeded in `e.code() == Code::DeadlineExceeded`: error=Status { code: Cancelled, message: "Timeout expired", source: Some(tonic::transport::Error(Transport, TimeoutExpired(()))) } ``` Seems like it failed like you said...

The error messages in transport are unfortunately not totally complete in this way. This was due to a few reasons, mostly, being that I didn't want the current code to...

It would be helpful if you could potentially run a git bisect to see which commit introduced this if its quite reproducible.

CI is failing due to ``` error: unused import: `compile_settings::CompileSettings` --> tonic-build/src/lib.rs:101:16 | 101 | pub(crate) use compile_settings::CompileSettings; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `-D unused-imports` implied by `-D warnings` =...

> Is it possible to set up a branch on tonic to tackle each of the workspace members? Yes, in fact I will just branch off tonic now and we...

Ok I created a https://github.com/hyperium/tonic/tree/next branch, @allan2 if you want to target this branch with this work that would be great. I am happier to do smaller PRs. That said...

> Edit: Are we positive that we should be using the replacement body struct over using anything that implements hyper::body::Body? It just feels like being generic over the body is...

I would like to drop a quick update from my part, obviously this upgrade is quite complicated due to how dependent tonic is on hyper for good and bad reasons....

@matze Nice! I think we could potentially offer this as an opt-in during codegen? I don't think there is anything stopping us from support both since its mostly codegen and...

> @LucioFranco I tested but the initial caveat still holds, all crates using pre-generated services require either async_trait or a nightly compiler. And emitting code with the feature flag check...