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

Already on GitHub? Sign in to your account

Small performance fix on setter with expire time. (1 call instead of 2) #129

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants

frail commented Mar 31, 2012

Instead of setting then arranging expire time used redis command setex (which sets with expire).

Owner

beberlei commented Mar 31, 2012

Can you refactor using two return statements and no else? Otherwise ok

Member

deeky666 commented Dec 3, 2013

This patch was applied in doctrine/cache@c7fd3e1

@deeky666 deeky666 closed this Dec 3, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment