Skip to content

feat(NODE-6274): add CSOT support to bulkWrite #4250

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

nbbeeken
Copy link
Contributor

Description

What is changing?

  • Adds CSOT support to bulkWrite
Is there new documentation needed for these changes?

No

What is the motivation for this change?

CSOT for all operations

Double check the following

  • Ran npm run check:lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@nbbeeken nbbeeken force-pushed the NODE-6274-bulkCSOT branch 2 times, most recently from d463822 to 416ee1a Compare September 27, 2024 20:22
@nbbeeken nbbeeken marked this pull request as ready for review September 27, 2024 20:22
@W-A-James W-A-James self-assigned this Sep 30, 2024
@W-A-James W-A-James self-requested a review September 30, 2024 14:51
@W-A-James W-A-James added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Sep 30, 2024
@nbbeeken nbbeeken requested a review from W-A-James September 30, 2024 17:54
@W-A-James W-A-James added Team Review Needs review from team and removed Primary Review In Review with primary reviewer, not yet ready for team's eyes labels Sep 30, 2024
@aditi-khare-mongoDB aditi-khare-mongoDB self-requested a review October 1, 2024 15:05
@W-A-James
Copy link
Contributor

Note: Test failures here are a result of known and unrelated KMS server issue which will be addressed in NODE-6405

@W-A-James W-A-James merged commit 09f6d7d into NODE-6090 Oct 2, 2024
19 of 27 checks passed
@W-A-James W-A-James deleted the NODE-6274-bulkCSOT branch October 2, 2024 21:32
baileympearson added a commit that referenced this pull request Oct 10, 2024
Co-authored-by: Bailey Pearson <bailey.pearson@mongodb.com>
baileympearson added a commit that referenced this pull request Oct 10, 2024
Co-authored-by: Bailey Pearson <bailey.pearson@mongodb.com>
nbbeeken added a commit that referenced this pull request Oct 14, 2024
Co-authored-by: Bailey Pearson <bailey.pearson@mongodb.com>
baileympearson added a commit that referenced this pull request Oct 21, 2024
Co-authored-by: Bailey Pearson <bailey.pearson@mongodb.com>
nbbeeken added a commit that referenced this pull request Nov 1, 2024
Co-authored-by: Bailey Pearson <bailey.pearson@mongodb.com>
nbbeeken added a commit that referenced this pull request Nov 5, 2024
Co-authored-by: Bailey Pearson <bailey.pearson@mongodb.com>
dariakp pushed a commit that referenced this pull request Nov 6, 2024
Co-authored-by: Bailey Pearson <bailey.pearson@mongodb.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Team Review Needs review from team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants