Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: add an example of a bidirectional gRPC call via tonic #404

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lunar-debian
Copy link

@lunar-debian lunar-debian commented Feb 1, 2021

Hi! This is an attempt to add an example of using actix and tonic together (#348) by integrating a client bidirectional gRPC call with other actors.

The proto provides an Echo service with an Echo RPC. It’s a bidirectional stream, sending EchoRequest and received EchoReply. I haven’t written the corresponding server yet, but it should be easy enough.

Anyway, this doesn’t build at the moment and I’m not sure to be on the right track…

The current idea is to have an EchoService, looking like gRPC, responsible for maintaining a connection to the gRPC server.
It accepts a RunEcho message to call the RPC and start an EchoRpc actor that would handle input/output for the bidirectional stream. Does it make sense to architecture things this way?

Where I’m stuck is how to implement the actual call to the RPC (see the RunEcho handler). I can’t find the proper way to integrate the call without lifetime issues. Any idea?

I'm quite new to Rust, so I would be thankful for any other comments.

This PR is also missing a README, which I would happily write after a validation of the general architecture by someone more knowledgeable with Actix and the general ecosystem.

@robjtede robjtede marked this pull request as draft February 1, 2021 14:08
@robjtede
Copy link
Member

robjtede commented Mar 6, 2022

If there's still interest in getting this PR in a state to merge we'd be interested. Otherwise, if there's no updates, I'll close it out in a couple weeks or so.

@robjtede robjtede linked an issue Mar 6, 2022 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tonic example
2 participants