Closed meganemura closed 6 years ago
Rails のマイグレーションファイルに使われる change_table メソッドに { bulk: true } のオプションをつけることができます。 このオプションを利用すると 1 つの ALTER TABLE クエリにまとめてくれるので特に理由がなければ使った方がいいと思っています。
change_table
{ bulk: true }
ALTER TABLE
このオプションは MySQL (ActiveRecord::ConnectionAdapters::AbstractMysqlAdapter) のみサポートされていて、他の Adapter では無視されます。
ActiveRecord::ConnectionAdapters::AbstractMysqlAdapter
この change_table メソッドに対して {bulk: true} オプションを付けることを推奨するような Cop があるといいんじゃないかなあと思っています。
{bulk: true}
プロジェクト内で強制するならモンキーパッチで change_table に常にオプション渡す、とかできると思いますが、RuboCop で検知できてもいいかもしれないと思ったので起票してみます。
Rails のマイグレーションファイルに使われる
change_table
メソッドに{ bulk: true }
のオプションをつけることができます。 このオプションを利用すると 1 つのALTER TABLE
クエリにまとめてくれるので特に理由がなければ使った方がいいと思っています。このオプションは MySQL (
ActiveRecord::ConnectionAdapters::AbstractMysqlAdapter
) のみサポートされていて、他の Adapter では無視されます。この
change_table
メソッドに対して{bulk: true}
オプションを付けることを推奨するような Cop があるといいんじゃないかなあと思っています。プロジェクト内で強制するならモンキーパッチで
change_table
に常にオプション渡す、とかできると思いますが、RuboCop で検知できてもいいかもしれないと思ったので起票してみます。