-
Notifications
You must be signed in to change notification settings - Fork 450
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Present a explanation box recommending to use CONCURRENTLY with DROP INDEX. In Active Record, this requires two things: disable_ddl_transaction! once at the top of the migration file, and "algorithm: :concurrently" option added to every remove_index line Since this is a sensible default when PostgreSQL 14 or greater is in use, since it can avoid downtime from an index drop, make it the default. Remove the ability to disable it since it seems unnecessary in retrospect. Only show the explanation box and only add the option to remove_index when version 14 or greater is in use
- Loading branch information
1 parent
8c42de4
commit 67fd867
Showing
4 changed files
with
21 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters