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

Motor uses ThreadPoolExecutor, it drastically decreases a performance #464

Open
nikita-davydov opened this issue Apr 28, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@nikita-davydov
Copy link

Is there any plan to make this library really async?

It seems like that nobody who uses async motor understand that it's not really async.

@nikita-davydov nikita-davydov added the enhancement New feature or request label Apr 28, 2024
@nikita-davydov nikita-davydov changed the title Motor uses ThreadPoolExecutor, it drastically decreases the perfomance Motor uses ThreadPoolExecutor, it drastically decreases a performance Apr 28, 2024
@z0z0r4
Copy link

z0z0r4 commented Apr 29, 2024

More information? This package uses motor by threading?

@nikita-davydov
Copy link
Author

nikita-davydov commented Apr 29, 2024

Just look into the sources of the official motor library: https://github.com/mongodb/motor/blob/master/motor/metaprogramming.py#L25

Any project that will use that for their requests to the mongoDB will have the performance decreased for sure.

@z0z0r4
Copy link

z0z0r4 commented May 1, 2024

Just look into the sources of the official motor library: https://github.com/mongodb/motor/blob/master/motor/metaprogramming.py#L25

Any project that will use that for their requests to the mongoDB will have the performance decreased for sure.

Emmm, I have not enough ability to confirm is it true...

The cost is too much to achieve and maintain an unofficial async package as same as pymongo...

Any response in motor jira about this problem?

@nikita-davydov
Copy link
Author

nikita-davydov commented May 9, 2024

They just have the dead JIRA ticket about that so seems like they're not going to implement the real asyncio connection

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants