crache icon indicating copy to clipboard operation
crache copied to clipboard

Delay returned on cache lookup

Open emidln opened this issue 10 years ago • 2 comments

I'm probably just missing something here, but why does lookup return a Delay? It's pretty frustrating to need to check delay? or force just for crache when other core.cache backends don't return Delays.

emidln avatar Dec 04 '14 23:12 emidln

I'm afraid that this was an oversight on my part- the delay were at one time optional (so that you could ask for a lookup before you needed, potentially an optimization), but I did not intend to return delays exclusively. I think this was brought in by #7, which included several other changes that seem to work well. I'm a bit busy at the moment, but I would accept a PR to re-add the option for delays.

strongh avatar Dec 05 '14 01:12 strongh

I fixed this in my fork: https://github.com/behrica/crache and merged as well 2 PRs: #15 #16

@strongh Do you want to include this changes in here ?

behrica avatar Jun 30 '23 16:06 behrica