-
-
Notifications
You must be signed in to change notification settings - Fork 856
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
Switch to tokio, and update dependencies #589
base: master
Are you sure you want to change the base?
Conversation
Thank you so much for this! We take performance seriously here. I'll have to read up more on Tokio vs STD:ASYNC but I think you're right, Tokio in my opinion would be faster now-a-days :) |
opps forgot to make a new branch, sorry for the extra pushes |
I'd like to add some things regarding the arguments in issue #588. It's the PR related to it, and I figured we could keep the discussion in one place. Considering performance, the scan engine was built on top of |
implement the #588 suggestion