-
Notifications
You must be signed in to change notification settings - Fork 20
DOCSP-43420: Client bulk write #177
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-43420: Client bulk write #177
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.
A couple things:
source/write/bulk-write.txt
Outdated
- Namespace in which to perform the update. This argument is optional if | ||
you perform the bulk operation on a single collection. | ||
- **Query filter** that specifies the criteria used to match documents in your collection. | ||
- Update operation you want to perform. For more information about update | ||
operations, see the :manual:`Field Update Operators | ||
</reference/operator/update-field/>` guide in the {+mdb-server+} manual. |
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.
You could do this or make this a table:
- Namespace in which to perform the update. This argument is optional if | |
you perform the bulk operation on a single collection. | |
- **Query filter** that specifies the criteria used to match documents in your collection. | |
- Update operation you want to perform. For more information about update | |
operations, see the :manual:`Field Update Operators | |
</reference/operator/update-field/>` guide in the {+mdb-server+} manual. | |
- Namespace: Where to perform the update. This argument is optional if | |
you perform the bulk operation on a single collection. | |
- Query filter: Specifies the criteria used to match documents in your collection. | |
- Update operation: The operation you want to perform. For more information about update | |
operations, see the :manual:`Field Update Operators | |
</reference/operator/update-field/>` guide in the {+mdb-server+} manual. |
Same of the proceeding operation parameters.
When {+driver-short+} runs a bulk operation, it uses the ``write_concern`` of the | ||
collection or client on which the operation is running. You can also set | ||
a write concern for the operation when using the ``MongoClient.bulk_write()`` | ||
method. The driver reports all write concern errors after attempting all operations, | ||
regardless of execution order. | ||
|
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.
Add link to Write Concern in the server manual
source/write/bulk-write.txt
Outdated
operation. If you don't specify any additional options, the driver does not customize | ||
the bulk write operation. |
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.
I realize you didn't add this, but it's an odd thing to explicitly call out, so I think we should remove it.
operation. If you don't specify any additional options, the driver does not customize | |
the bulk write operation. | |
operation. |
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.
One tiny suggestion.
source/write/bulk-write.txt
Outdated
|
||
You can use bulk operations to perform multiple write operations on | ||
a collection. You can also call bulk operations on your client, | ||
which allows you to perform bulk writes across multiple namespaces. |
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.
I'd probably say, "Both collections and clients have bulk_write methods. Using the latter allows you to perform bulk writes across multiple namespaces."
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.
I reworded, let me know what you think!
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!
* DOCSP-43420: Client bulk write * versions * output * error type * wording * RM feedback * admonition * CC feedback (cherry picked from commit 7609c5a)
* DOCSP-43420: Client bulk write * versions * output * error type * wording * RM feedback * admonition * CC feedback (cherry picked from commit 7609c5a)
* DOCSP-43420: Client bulk write * versions * output * error type * wording * RM feedback * admonition * CC feedback (cherry picked from commit 7609c5a)
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-43420
Staging Links
Self-Review Checklist