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

Using uvloop #4369

Closed
jakirkham opened this issue Dec 16, 2020 · 3 comments · Fixed by #4448
Closed

Using uvloop #4369

jakirkham opened this issue Dec 16, 2020 · 3 comments · Fixed by #4448

Comments

@jakirkham
Copy link
Member

I'm curious to know if we have looked into using uvloop with Distributed recently (particularly on the Scheduler). There are some older threads that mention this, but some of them are 2-3 years old. Here's one on the worker ( #2156 ), communication ( #2162 ) ( #3145 ), and some others can be found as well (usually thinking about communication). What I'm more interested in knowing than the communication bit, is if having enough of the async bits Cythonized ends up helping overall Scheduler runtime?

@jakirkham
Copy link
Member Author

Occasionally Sanic comes up in this context as an alternative to Tornado. Though IIUC outside of using uvloop, which Tornado could also use, it uses httptools to parse URLs in C, which is faster, but probably not very important to us (unless I'm missing something). So not sure a Sanic vs. Tornado comparison would be interesting, but could be wrong about that.

@mrocklin
Copy link
Member

mrocklin commented Dec 16, 2020 via email

@jakirkham
Copy link
Member Author

This is being addressed in PR ( #4448 )

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 a pull request may close this issue.

2 participants