Skip to content
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

Remove redundant indexes created by the default migration files #1092

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JewelSam
Copy link
Contributor

Context
Several redundant indexes are created by the default migration files generated
Issue: #1083

Solution

  • remove index_taggings_on_tag_id - can be replaced by taggings_idx
  • remove index_taggings_on_taggable_id - can be replaced by taggings_idy or taggings_taggable_context_idx
  • remove index_taggings_on_taggable_type - can be replaced by index_taggings_on_taggable_type_and_taggable_id
  • remove index_taggings_on_tagger_id - can be replaced by index_taggings_on_tagger_id_and_tagger_type

- remove index_taggings_on_tag_id - can be replaced by taggings_idx
- remove index_taggings_on_taggable_id - can be replaced by taggings_idy or taggings_taggable_context_idx
- remove index_taggings_on_taggable_type - can be replaced by index_taggings_on_taggable_type_and_taggable_id
- remove index_taggings_on_tagger_id - can be replaced by index_taggings_on_tagger_id_and_tagger_type
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant