Skip to content

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

Merged
merged 2 commits into from
May 16, 2022

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented May 3, 2022

@jmikola jmikola force-pushed the cdriver-4199 branch 2 times, most recently from 294818c to 5c959d1 Compare May 4, 2022 18:18
@jmikola jmikola marked this pull request as ready for review May 4, 2022 19:42
@jmikola jmikola requested review from eramongodb and kevinAlbs May 4, 2022 19:42
Copy link
Collaborator

@kevinAlbs kevinAlbs left a 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.

@kevinAlbs kevinAlbs self-requested a review May 9, 2022 17:23
Copy link
Collaborator

@kevinAlbs kevinAlbs left a 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.

Copy link
Contributor

@vector-of-bool vector-of-bool left a 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.

jmikola and others added 2 commits May 12, 2022 21:10
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]>
@jmikola
Copy link
Member Author

jmikola commented May 13, 2022

@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.

@jmikola jmikola merged commit 29da448 into mongodb:master May 16, 2022
@jmikola jmikola deleted the cdriver-4199 branch May 16, 2022 17:04
@jmikola jmikola mentioned this pull request May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants