Add pattern & storage cache factory #3056

Closed
wants to merge 2 commits into
from

Projects

None yet

3 participants

@blanchonvincent
Contributor

Make life easier with a default storage & pattern cache factory for the service manager.

@marc-mabe
Member

It doesn't make sense to have one default cache pattern -- each pattern solves a different problem. So there should be one default PatternFactory for each configurable cache pattern. Shouldn't it ?

@blanchonvincent
Contributor

Hmm, yeah you are right. I often use the same pattern but yes, each pattern must be have a factory.

@blanchonvincent
Contributor

@marc-mabe cache pattern factory has been removed, it's better :)

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