-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Misc MongoDB Connector Fixes #5877
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5877 +/- ##
==========================================
+ Coverage 66.89% 66.93% +0.03%
==========================================
Files 130 130
Lines 4151 4159 +8
Branches 637 640 +3
==========================================
+ Hits 2777 2784 +7
+ Misses 999 997 -2
- Partials 375 378 +3
Continue to review full report at Codecov.
|
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.
Nice! This is actually a very significant enhancement and turns on a lot of Mongos features in this connector.
Description
Rebase of #5875 for @melohagan
Added support for ObjectIds. Made some fixes for the CRUD operations.
Addresses:
There's still a number of improvements to be done in the V2 connector, but this should help a bit in the meantime.
Screenshots
Insert
Update