-
Notifications
You must be signed in to change notification settings - Fork 10.9k
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
chore: remove redundant indexes #30914
Conversation
|
Don't we need migrations to actually remove those indexes from mongo? |
yes, we'd need a migration to remove them.. we're avoiding migrations during minor releases though, so we'd need to "schedule" a migration for the next major.. |
Codecov Report
@@ Coverage Diff @@
## develop #30914 +/- ##
===========================================
- Coverage 51.42% 51.12% -0.30%
===========================================
Files 814 813 -1
Lines 15046 15232 +186
Branches 2732 2814 +82
===========================================
+ Hits 7737 7788 +51
- Misses 6902 7007 +105
- Partials 407 437 +30
Flags with carried forward coverage won't be shown. Click here to find out more. |
already created this as a follow up #30918 |
Proposed changes (including videos or screenshots)
the indexes removed are actually redundant and are prefixes of other compound indexes.
Unused indexes consume storage space and increase write latency.
Issue(s)
Steps to test or reproduce
Further comments