-
Notifications
You must be signed in to change notification settings - Fork 407
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
Fix validation process for creating duplicate _id
index
#3990
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3990 +/- ##
==========================================
+ Coverage 74.53% 74.59% +0.06%
==========================================
Files 337 337
Lines 27015 27015
==========================================
+ Hits 20136 20153 +17
+ Misses 5623 5610 -13
+ Partials 1256 1252 -4
... and 4 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
The test looks good to me, and the fix is also reasonable.
However, now an SQLite test task test-integration-sqlite TEST_RUN='TestCreateIndexesCompat'
fails, so we need some more fixes. Please feel free to ping me if you have any questions about it.
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.
Ok, I see all index-related tests pass, so the fix works!
_id
_id
index
Description
Align behavior of index creation commands for key:
_id
to meet MongoDB requirements -those operations should not raiseIndex already exists with a different name
error when key:_id
(no matter if there isname
in the query).Closes #3921.
Readiness checklist
task all
, and it passed.@FerretDB/core
), Milestone (Next
), Labels, Project and project's Sprint fields.