CSV出力時にメモリを使い切ってしまう問題を修正

[chihiro-adachi]

概要(Overview・Refs Issue)

#4775 の対応
商品・カテゴリ・受注・会員CSVを対応しました。

方針(Policy)

trigger_errorが発生するメソッドを利用しないように修正

実装に関する補足(Appendix)

EntityManager::clearメソッドを利用した場合、ManyToOneの出力(性別など)が欠落する問題が発生したため、UnitOfWork::detachを利用するようにしています。※EntityManager::detachと挙動は同じ

01d6b15

テスト（Test)

• memory_limit 128MBで商品点数 15,262件/15,262件を出力できることを確認

相談（Discussion）

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

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

レビュワー確認項目

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

[nanasess]

EntityManager::clearメソッドを利用した場合、ManyToOneの出力(性別など)が欠落する問題が発生したため、UnitOfWork::detachを利用するようにしています。※EntityManager::detachと挙動は同じ

detach は deprecated になっているので、 clear で何とかなりませんかね。。。

[chihiro-adachi]

@nanasess
clear(entity_name)が使えればよいのですが、そちらもdeprecatedでtrigger_errorが発生します。

もしくは、以下のようにページングしてclear()するのが適切かと思いましたが、受注と受注明細のようなクエリだと件数の制御がうまくいかず、ちょっと手詰まりな感じです。
http://enterprisegeeks.hatenablog.com/entry/2014/11/24/110723

いつかは修正しないといけない問題ではあるんですが、4.0.xではぎりぎり許容範囲かなあと。。

何かよいやり方があればアドバイスいただけるとありがたいです。

[chihiro-adachi]

@nanasess
ページングしてやればよさげでした。
以下のように修正してみましたがどうでしょうか。
0da7c07

本PRの適用前/後でCSVの差分を比較して差異がないことも確認済。

[nanasess]

@chihiro-adachi 良いと思います。ありがとうございます！