Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

設定のキャッシュを Symfony Config Component の Cache に移行 #1897

Closed
ttsuru opened this issue Oct 31, 2016 · 5 comments
Closed

Comments

@ttsuru
Copy link
Contributor

ttsuru commented Oct 31, 2016

Symfony\Component\Config\ConfigCache にてキャッシュを行うことで、独自実装が不要となるため、どこかのタイミングで移行を検討できればと思います。

refs
http://symfony.com/doc/2.7/components/config/caching.html
http://api.symfony.com/2.7/Symfony/Component/Config/ConfigCache.html

isFresh により更新を確認し、自動更新を実装可能です。

@nanasess
Copy link
Contributor

Symfony\Component\Config\ConfigCache の使用も検討していましたが、

上記のような理由で、独自実装となっています。
Symfony\Component\Config\ConfigCache で解決可能な課題であれば、移行して問題ないと思います。

@ttsuru
Copy link
Contributor Author

ttsuru commented Oct 31, 2016

@nanasess

情報ありがとうございます。

#1695 キャッシュではなく、 config ファイルを PHP で記述できるようにする対応

https://github.com/symfony/config/blob/2.7/ConfigCache.php#L120
こちらはserializeだと速度的な問題がありそうなのでしょうか。

@nanasess
Copy link
Contributor

@ttsuru
僅差みたいですね。 http://blog.tojiru.net/article/133500110.html
laravel でも、 config は PHP ファイルですし、short array syntax 使えば可読性も悪くないので、わざわざ Yaml にしてキャッシュするまでもないかなぁと思っています。

@ttsuru
Copy link
Contributor Author

ttsuru commented Oct 31, 2016

@nanasess
本件、ずばり探していたソースでした。ありがとうございます。
Symfony で var_export を使わずに serialize を利用している理由は後ろの方にある外部ファイルにした場合のパフォーマンスっぽいですね。

isFresh を利用できれば、キャッシュの確認などをプラグイン側で意識しなくても良さそうなので、良い実装があれば移行できると嬉しいです。

@Yangsin Yangsin changed the title 設定のキャッシュを Symfony Config Component の Cache に以降 設定のキャッシュを Symfony Config Component の Cache に移行 Oct 31, 2016
@Yangsin Yangsin added this to the 3.x milestone Oct 31, 2016
@Yangsin Yangsin modified the milestones: 3.x, 3.n.0, Not release Oct 17, 2017
@Yangsin
Copy link

Yangsin commented Oct 17, 2017

3.nではYaml自体を廃止することとしましたので、クローズいたします。

@Yangsin Yangsin closed this as completed Oct 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants