-
Notifications
You must be signed in to change notification settings - Fork 20
DOCSP-45883 - Parallelism in transactions #135
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-45883 - Parallelism in transactions #135
Conversation
✅ Deploy Preview for docs-pymongo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
methods described in the preceding section to manually manage the transaction lifecycle. | ||
methods described in the preceding section to manually manage the transaction lifecycle. | ||
|
||
.. note:: Parallel Operations Not Supported |
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.
need to add note manually because {+driver-short+} = 'PyMongo'
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.
Left a question, so requesting changes for that!
|
||
.. note:: Parallel Operations Not Supported | ||
|
||
{+driver-short+} does not support running parallel operations within a single transaction. |
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.
Q: does PyMongo not support bulk operations on multiple namespaces? If so, I'd add that to the admonition and link to the Bulk Write guide
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.
Versions 4.9+ do, but we haven't documented it yet, so there's nothing to link to. (Could link to the old docs domain, but that seems like it could be confusing for people...?)
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.
Ah okay! then I think not linking is fine
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!
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
(cherry picked from commit 5ea0b78)
(cherry picked from commit 5ea0b78)
(cherry picked from commit 5ea0b78)
(cherry picked from commit 5ea0b78)
(cherry picked from commit 5ea0b78) Co-authored-by: Mike Woofter <[email protected]>
(cherry picked from commit 5ea0b78) Co-authored-by: Mike Woofter <[email protected]>
(cherry picked from commit 5ea0b78) Co-authored-by: Mike Woofter <[email protected]>
(cherry picked from commit 5ea0b78) Co-authored-by: Mike Woofter <[email protected]>
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-45883
Staging - https://deploy-preview-135--docs-pymongo.netlify.app/write/transactions/#example
Self-Review Checklist