大きなサイズのCSVを扱えるようタイムアウトを延長

[nanasess]

概要(Overview・Refs Issue)

src/Eccube/Resource/doctrine/import_csv 以下に、独自の初期化用CSV ファイルを配置することで、インストール時のデータを独自にカスタマイズ可能。
また、効率的にデータインポート可能なので、 memory_limit を越えたサイズの CSV ファイルも扱うことができる。

しかし大きなサイズの CSV ファイルを登録すると max_execution_time に抵触し、システムエラーになってしまう

方針(Policy)

set_time_limit() 関数でタイムアウトを延長する。

実装に関する補足(Appendix)

2系で実装されている、以下のコードを流用
https://github.com/EC-CUBE/ec-cube2/blob/4929cf1d0e4ab0b76cd4b3201998e72eeca2ccfb/data/class/util/SC_Utils.php#L1840-L1864

テスト（Test)

3,145,724行の mtb_country.csv を作成し、 memory_limit=128M の環境でインストール完了することを確認

ls -al | grep mtb_country
.rw-r--r--  128M nanasess  8 6 19:36  mtb_country.csv
wc -l mtb_country.csv
 3145725 mtb_country.csv

select count(*) from mtb_country;
+----------+
| count(*) |
+----------+
| 3145724  |
+----------+

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

ありがとうございます。
落ちているテストは今回の修正に起因するものではないのでスルーでOKかとおもいます。

[okazy]

ありがとうごじます！OKです。

[okazy]

ありがとうございます。マージしました。