bearsaturday / BEAR.Saturday

PHP 5.2+ resource-oriented web framework
https://github.com/bearsaturday/manual
Other
20 stars 16 forks source link

BEAR_Page BEAR_Main のgetCacheKey() #31

Closed usomillp closed 10 years ago

usomillp commented 10 years ago

memcacheを使っているサイトでBEAR_Page BEAR_Main のgetCacheKey() でmemcacheのkeyサイズの最大の250バイトを超えるケースがあったのですが、 これは BEAR_Cache_Adapter_Memcache の set , get 側を直すべきでしょうか? それとも 呼び元のBEAR_Page BEAR_Mainを直すべきかどちらがよいですかね?

memcacheに依存する問題なので、BEAR_Cache_Adapter_Memcache のset getのところでキーをhash化すべき? ただ Roのキャッシュは BEAR_Ro_Prototypeでmd5にしていたので…

どうでしょうか?

koriym commented 10 years ago

元々はmd5を使ってなかったのは下記のような理由なのですが、250バイトを超えるケース こうなってくると話は別です。

pushしたようなBEAR_Pageの単純な修正でどうでしょうか。

usomillp commented 10 years ago

なるほど、そっちのほうがいまのやつはシンプルでいいと思います。

koriym commented 10 years ago

了解です。mergeしてdevelopにpushしました。 ありがとうございました。