OpenPNE3 Plugins » opAshiatoPlugin

この問題を解決できるであろうパッチを作成しました。

さらに追加でパッチを作成しました。

日時が誤っている問題の修正です。

#note-3 にてついてはすでに修正されているので適応する必要はないです。

#note-4 は現在の最新版(0.9.1.2)でも修正されておらず、順序が間違って表示されています。
意図的に再現させた際のスクリーンショットを添付しておきます。

修正方針ですが、
添付されているパッチでは、updated_at の名前を変更して対応を行なっていますが、
この場合修正範囲が広くなるため別の対応で行いたいです。
「->orderBy('MAX(updated_at) DESC');」 のように名前を変更せずに
MAX関数を追加する方向で修正するほうが修正範囲が少なく安全ではないかと思われます。
（負荷的な面ではこちらのほうが重い？）

https://github.com/tejimaya/opAshiatoPlugin/commit/7e8f42939471abf54a9ad273d695e4a84e97f1f6 にてコミットしました。

「->orderBy('MAX DESC');」 のように名前を変更せずにMAX関数を追加する方向で修正

でも名前を変更した場合と同等の負荷になるようですが、ここでは（今のまま）名前の変更で問題ありません。

他の箇所でも参照する変数であるため、"ua" より説明的な変数名（例："max_updated_at"）への変更が望ましいです。

備考¶

MySQLのEXPLAIN文で見る限りでは負荷要因の軽減は見られませんでした：

mysql> EXPLAIN SELECT *,MAX(updated_at) as updated_at FROM ashiato WHERE member_id_to = 3 AND r_date IN ('2011-05-19') GROUP BY member_id_from, r_date ORDER BY MAX(updated_at) DESC;
+----+-------------+---------+------+-------------------------------------------------------------------------------------+-------------------------+---------+-------------+------+----------------------------------------------+
| id | select_type | table   | type | possible_keys                                                                       | key                     | key_len | ref         | rows | Extra                                        |
+----+-------------+---------+------+-------------------------------------------------------------------------------------+-------------------------+---------+-------------+------+----------------------------------------------+
|  1 | SIMPLE      | ashiato | ref  | member_id_to_idx,member_id_to_member_id_from_updated_at_idx,member_id_to_r_date_idx | member_id_to_r_date_idx | 9       | const,const |    1 | Using where; Using temporary; Using filesort | 
+----+-------------+---------+------+-------------------------------------------------------------------------------------+-------------------------+---------+-------------+------+----------------------------------------------+
1 row in set (0.06 sec)

mysql> EXPLAIN SELECT *,MAX(updated_at) as max_updated_at FROM ashiato WHERE member_id_to = 3 AND r_date IN ('2011-05-19') GROUP BY member_id_from, r_date ORDER BY max_updated_at DESC;
+----+-------------+---------+------+-------------------------------------------------------------------------------------+-------------------------+---------+-------------+------+----------------------------------------------+
| id | select_type | table   | type | possible_keys                                                                       | key                     | key_len | ref         | rows | Extra                                        |
+----+-------------+---------+------+-------------------------------------------------------------------------------------+-------------------------+---------+-------------+------+----------------------------------------------+
|  1 | SIMPLE      | ashiato | ref  | member_id_to_idx,member_id_to_member_id_from_updated_at_idx,member_id_to_r_date_idx | member_id_to_r_date_idx | 9       | const,const |    1 | Using where; Using temporary; Using filesort | 
+----+-------------+---------+------+-------------------------------------------------------------------------------------+-------------------------+---------+-------------+------+----------------------------------------------+
1 row in set (0.00 sec)

https://github.com/tejimaya/opAshiatoPlugin/commit/eb999df3b00cd503ec61cffc93227ac30477c892 にて指摘点の修正を行いました。

OKです。修正対応どうもありがとうございます。

テスト完了いたしました。
問題は発見されなかったためFixedいたします。