Skip to content

Remove old vae and eav indexes#1330

Merged
dwwoelfel merged 2 commits intomainfrom
remove-indexes
Jul 31, 2025
Merged

Remove old vae and eav indexes#1330
dwwoelfel merged 2 commits intomainfrom
remove-indexes

Conversation

@dwwoelfel
Copy link
Contributor

@dwwoelfel dwwoelfel commented Jul 2, 2025

Removes the indexes that were replaced in #1326

I'll delay merging and check the autoexplain to be sure we don't still have queries relying on these indexes.

Deployment plan

  • Merge to main
  • Drop the indexes concurrently
drop index concurrently if exists vae_index;

drop index concurrently if exists eav_index;
  • Run the migrations

@github-actions
Copy link
Contributor

github-actions bot commented Jul 2, 2025

View Vercel preview at instant-www-js-remove-indexes-jsv.vercel.app.

Copy link
Contributor

@stopachka stopachka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heeck yeah!

@dwwoelfel dwwoelfel merged commit 20262bb into main Jul 31, 2025
33 checks passed
@dwwoelfel dwwoelfel deleted the remove-indexes branch July 31, 2025 17:38
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.

2 participants