Upgrade to Tokio 1.0.0 ecosystem

[QuentinPerez]

Hey 👋 ,

A quick PR to finish the work done by @marshaled here #514.

Before merging, we have to wait for the release of warp, this PR depends on a fork.

[coolreader18]

It looks like tower-balance and some other crates have been merged into tower submodules gated by features (which you've enabled). The current tower documentation says:

Tower is made up of the following crates:

• tower (this crate)
• tower-service
• tower-layer
• tower-test

So I think it would be good to drop the other crates, which still depend on tokio 0.2 as well.

[QuentinPerez]

Oh yes good catch, fixed thx

[simonsan]

• warp Upgrade to Tokio 1.0 seanmonstar/warp#753 (soon to be merged)

[davidpdrsn]

I've tried updating my codebase to use this patch and ran into something a little annoying. It seems Channel is not longer Sync because it contains a Semaphore from tower. Channel being Sync is something I relied upon in my codebase. Not sure if I should find a solution that doesn't depend on Channel being Sync or if there is some way to rectifying it.

[coolreader18]

@davidpdrsn I'm not sure how that'd be useful for Channel to be Sync, since all the uses of it require &mut anyway. Maybe try putting it in a Mutex to make it Sync? Since I assume you just need to store it if it was working before with mutable access, you wouldn't ever need to lock it, and you can just use Mutex::get_mut() when you do have mut access.

[LucioFranco]

Thanks! Will get this merged so we can move forward.

[LucioFranco]

This is just for examples, so not blocking a release!

[davidpdrsn]

@coolreader18 its being fixed in tower as we speak 😊

[edrevo]

Are there any other changes needed for a new release of tonic? Can't wait to get all the good bits that have been added since 0.3.1!

[LucioFranco]

@edrevo going through that all right now :)