GODRIVER-3668 Add bypassEmptyTsReplacement option. #2219
+656
−3
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🧪 Performance ResultsCommit SHA: 446c2e1The following benchmark tests for version 68e51cf3f83d2a00074816f5 had statistically significant changes (i.e., |z-score| > 1.96):
For a comprehensive view of all microbenchmark results for this PR's commit, please check out the Evergreen perf task for this patch. |
API Change Report./v2/x/mongo/driver/operationcompatible changes(*FindAndModify).AdditionalCmd: added |
qingyang-hu
force-pushed
the
godriver3668
branch
from
f42d087 to
446c2e1
Compare
qingyang-hu
changed the title
There was a problem hiding this comment.
Pull Request Overview
This PR adds the bypassEmptyTsReplacement option to various MongoDB write operations by implementing an "addCommandFields" mechanism that allows arbitrary command fields to be added to operations.
- Adds support for bypassEmptyTsReplacement option across insert, update, replace, findOneAndUpdate, findOneAndReplace, and bulk write operations
- Implements addCommandFields functionality in internal options handling to support additional command parameters
- Adds comprehensive test coverage to verify the option is properly passed through to MongoDB commands
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| x/mongo/driver/xoptions/options.go | Adds "addCommandFields" case handling to all internal option setters |
| x/mongo/driver/operation/update.go | Adds additionalCmd field and AdditionalCmd method to Update operation |
| x/mongo/driver/operation/insert.go | Adds additionalCmd field and AdditionalCmd method to Insert operation |
| x/mongo/driver/operation/find_and_modify.go | Adds additionalCmd field and AdditionalCmd method to FindAndModify operation |
| mongo/collection.go | Integrates addCommandFields option handling into collection methods |
| mongo/client_bulk_write.go | Adds additionalCmd field to clientBulkWrite struct |
| mongo/client.go | Integrates addCommandFields option handling into Client.BulkWrite |
| mongo/bulk_write.go | Adds additionalCmd field and integrates it into bulk write operations |
| internal/integration/collection_test.go | Adds comprehensive tests for bypassEmptyTsReplacement option |
| internal/integration/client_test.go | Adds tests for client bulk write with bypassEmptyTsReplacement and fixes test assertions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
GODRIVER-3668
GODRIVER-3582
Summary
Add bypassEmptyTsReplacement option to:
Background & Motivation