SC_Helper_DB#sfCountCategory 抽出条件の重複指定の懸念

[Seasoft]

1つの SQL 文の構築で、$sql_where_product_ids を2回埋め込んでいる。

実際のところは、結合条件などを分析しないと断定できないが、冗長な記述となっている懸念を感じる。

→ 確認した範囲では、内側の問い合わせは、抽出結果に影響を与えていなかった。

おそらく、DB のオプティマイザが1度の処理に集約しているとは思うが、結合条件などによっては、2度の処理が走っている懸念もある。

→ 外側のみにしたところ、ある PostgreSQL 8.4 環境において、16% 程度の問い合わせコストの改善が見られた。(オプティマイザが弱いと結果が逆転する懸念はある。)

r21324

[Seasoft]

改訂を行いますが、もしも問題があるようでしたら、差し戻すなどした上で、ソースにコメントを残しておいて下さい。

[kotani]

2_11-devブランチのコミットキャンセルが必要

[Seasoft]

kotani への返信

2_11-devブランチのコミットキャンセルが必要

r21410 で対応しました。

[kajiwara]

Seasoftさん

本件、ご対応ありがとうございます。

こちらで、特に不具合もでていないようですので、こちらでクローズさせていただきます。