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

Create async version of tinyrpc #88

Open
lnoor opened this issue May 13, 2021 · 8 comments
Open

Create async version of tinyrpc #88

lnoor opened this issue May 13, 2021 · 8 comments
Assignees
Milestone

Comments

@lnoor
Copy link
Collaborator

lnoor commented May 13, 2021

No description provided.

@lnoor lnoor added this to the v1.1 milestone May 13, 2021
@lnoor lnoor self-assigned this May 13, 2021
@amitlissack
Copy link

amitlissack commented Oct 5, 2021

This would be great!

I see there's a branch. What is the status of this feature?

@lnoor
Copy link
Collaborator Author

lnoor commented Oct 6, 2021

Hi. Yes there is lnoor/asyncio based on the work of DrPyser and Dan Gusev.
As you can see it's been a while since I worked on it.

Status at the moment is that there is an async dispatcher and server. I'm working on async transports and their tests.
There is currently little to no progress. Other activities take precendence at the moment.

But it is good to know that people are still interested in this feature.
Is your interest primarily server or client side.
Although I have a couple of ideas about it, I haven't started work on the client side yet.
If you are willing to help out, I gladly share those ideas and see what comes of it.

@amitlissack
Copy link

I am very interested in client and server. I will take a peek at your branch and see if I can contribute.

@lnoor
Copy link
Collaborator Author

lnoor commented Oct 7, 2021

Please do.
If you want to work on it let's talk what needs to be done.
It would be regrettable if your work is not included.

@amitlissack
Copy link

Are you planning to create AsyncServerTransport derived versions of all the Transport classes?

@lnoor
Copy link
Collaborator Author

lnoor commented Oct 11, 2021

No, only those I expect to use myself: CallbackServerTransport and perhaps ZmqServerTransport or WsgiServerTransport.

@lnoor
Copy link
Collaborator Author

lnoor commented Oct 13, 2021

"Are you planning to create AsyncServerTransport derived versions of all the Transport classes?"
Is that perhaps a way to ask what I plan to do myself?
If so, you're welcome to work on the server part as well.

@amitlissack
Copy link

Not really. I was curious to know what your strategy was for supporting asyncio. Whether each client and server transport would need asyncio vs non-asyncio versions.

I apologize for not responding sooner. Busy time.

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

No branches or pull requests

2 participants