Skip to content

[移行-インポート] DBインデックス追加によるパフォーマンス改善#2125

Merged
akagane99 merged 1 commit into
masterfrom
migration
Feb 17, 2025
Merged

[移行-インポート] DBインデックス追加によるパフォーマンス改善#2125
akagane99 merged 1 commit into
masterfrom
migration

Conversation

@akagane99
Copy link
Copy Markdown
Contributor

@akagane99 akagane99 commented Feb 17, 2025

概要

移行のインポート時に、既に移行済みか確認するテーブル MigrationMapping を1件1件参照していますが、DBインデックスがありませんでした。
DBインデックスを追加したところ、参照速度が改善されましたので、プルリクエストを作成しました。

参考:MigrationMapping を1件1件参照

// マッピングテーブルの取得
$mapping = MigrationMapping::where('target_source_table', 'uploads')->where('source_key', $upload_key)->first();

SQL参照速度

SELECT * FROM `migration_mappings` WHERE target_source_table = 'uploads' and source_key = '11'; 

-- 行 0 - 0 の表示 (合計 1, クエリの実行時間: 0.0060 秒。) 
-- ↓index追加後
-- 行 0 - 0 の表示 (合計 1, クエリの実行時間: 0.0001 秒。) 

移行データ1件につき、1Selectしているため、
例えば10万件などの大量データの移行時、速度改善が期待できます。

レビュー完了希望日

なし

関連Pull requests/Issues

なし

参考

なし

DB変更の有無

有り

チェックリスト

@akagane99 akagane99 added the developer update 開発者向けの更新 label Feb 17, 2025
@akagane99 akagane99 self-assigned this Feb 17, 2025
@akagane99 akagane99 merged commit debc4b7 into master Feb 17, 2025
@akagane99 akagane99 deleted the migration branch February 17, 2025 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer update 開発者向けの更新

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant