TDiary::Plugin の多重初期化を減らしたい

[machu]

TDiary::Pluginの初期化回数を減らすことで数10%の高速化が可能。
ただし、@pluginの使い回しに対応していないプラグインが存在するので、そのままでは導入できない状態。

参考

• TDiary::Plugin の多重初期化を減らしたい
• blog-category.rbが動作しない
• all_filtersとかload_pluginsが呼ばれ過ぎで遅い件

[hsbt]

blog-category.rb をてこ入れして頑張りましょうか...

[tdtds]

まぁたしかにアレさえなんとかなれば……

[hsbt]

@machu これ、今月の 21 日くらいまでにできそうですかねえ。出来るなら 29 日リリースにいれたいところです。

[machu]

blog-category周りの問題をちゃんと理解していないので、29日リリースに含めるのは難しいです。
その次をターゲットにしたいです。

[hsbt]

手元にあった作業途中のブランチを tdiary/tdiary-core に持って来ました。

https://github.com/tdiary/tdiary-core/tree/memoization-load_plugins

[machu]

blog-category.rbだけの問題であれば、 #151 が対応できるまではBlogKitだけ無効化していったん導入してみたいです。

[hsbt]

良いと思います。

[machu]

参考までに、ローカル環境 (Mac OS X + ruby 1.9.3p362 + Thin 1.5.0) でのパッチ適用前後の速度比較です。
最後のカラムが応答時間です。おおよそ、20~30%高速化しています。

パッチ適用前

127.0.0.1 - - [16/Jan/2013 22:13:03] "GET / HTTP/1.0" 200 25967 0.4542
127.0.0.1 - - [16/Jan/2013 22:13:03] "GET / HTTP/1.0" 200 25967 0.1803
127.0.0.1 - - [16/Jan/2013 22:13:03] "GET / HTTP/1.0" 200 25967 0.1725
127.0.0.1 - - [16/Jan/2013 22:13:04] "GET / HTTP/1.0" 200 25967 0.1982

パッチ適用後

127.0.0.1 - - [16/Jan/2013 22:12:12] "GET / HTTP/1.0" 200 25967 0.3717
127.0.0.1 - - [16/Jan/2013 22:12:19] "GET / HTTP/1.0" 200 25967 0.1397
127.0.0.1 - - [16/Jan/2013 22:12:20] "GET / HTTP/1.0" 200 25967 0.1325
127.0.0.1 - - [16/Jan/2013 22:12:21] "GET / HTTP/1.0" 200 25967 0.1359

[machu]

blog-category.rb を導入していない環境ではキャッシュを有効にした → #242
#151 のコンテンツフィルタが実装できて blog-category.rb の多重初期化が不要になれば、 blog-category.rb 環境でもキャッシュを有効にすること。
（忘れないようにチケットはオープンのまま）

[hsbt]

この問題自体は解決しているので分岐処理を消すというのは #151 にマージした方がよさそうに見えます。

[machu]

#151 に追記したので、こちらはクローズします。