-
Notifications
You must be signed in to change notification settings - Fork 455
CDRIVER-4199: Add support for the comment field to all helpers #982
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
Conversation
294818c
to
5c959d1
Compare
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.
This looks good. Left two suggested test fixes. Thank you for calling out questions with comments and the organized commits.
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 with a warning fix.
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. Nothing sticks out, but the repetition in mongoc-opts.c
makes me curious about refactoring that file in particular.
Sync unified change stream and CRUD tests with mongodb/specifications@b8371ea Comment option for bulkWrite, insert, update, delete, findAndModify, aggregate, and change stream operations. Attach cursor comment to getMore for MongoDB 4.4+ Surface entity_map_match error if expectResult fails. The entity_map_match error contains relevant information about the failed match (e.g. field path). This is consistent with error reporting for error.expectResult later in the function. Update server doc links for database, collection, index enumeration. Also moves the retryable reads include for listDatabases to be consistent with other pages. Use BSON_ASSERT_PARAM for non-null parameter assertions in mongoc-bulk-operation.c. Co-authored-by: Kevin Albertson <[email protected]>
@kevinAlbs: Can you help me discern the actual build failure in debug-compile-nosasl-winssl? This is the only task whose build status is inconsistent with the base commit in https://spruce.mongodb.com/version/627db0e361837d34963b517d/tasks. Edit: just to follow-up, the build failure was unrelated so I proceeded with merging. |
https://jira.mongodb.org/browse/CDRIVER-4199
https://jira.mongodb.org/browse/CDRIVER-4374