-
Notifications
You must be signed in to change notification settings - Fork 17
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
DOCSP-48170: Async examples for Serialization/Third Party Tools #231
DOCSP-48170: Async examples for Serialization/Third Party Tools #231
Conversation
✅ Deploy Preview for docs-pymongo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
source/tools.txt
Outdated
_ = monkey.patch_all() | ||
from pymongo import MongoClient | ||
client = MongoClient() | ||
# You must call patch_all() *before* importing any other modules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The async API doesn't support using gevent
. That should be explicitly called out in the docs, where would the best place be to do so?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably either in the breaking changes page or on the forthcoming Sync vs. Async page. In the meantime, can undo this change here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it make sense to add a little callout here saying that AsyncMongoClient doesn't support gevent
?
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-48170
Staging Links
Self-Review Checklist