Adding the option to give out ttl if use_native_ttl is on in the MetaStore #73

Closed
wants to merge 2 commits into
from

Projects

None yet

3 participants

@rajatvig

No description provided.

@rajatvig

Fixes the issue #72

@rajatvig

Is this going to be fixed?

@mje113
Contributor
mje113 commented Oct 28, 2013

I was just about to open a pull request for the same issue. Perhaps there's a reason that I'm not seeing, but I would think that if the EntityStore for a given entry was purged at the cache level--due to ttl--the corresponding MetaStore entry should be allowed to be purged as well.

+1

@mje113
Contributor
mje113 commented Oct 28, 2013

Eh, went ahead with my almost-identical merge request since it has a test to cover the additional (optional) ttl arg on MetaStore#write.

#92

@grosser
Collaborator
grosser commented Oct 8, 2015

closing since there is an idential PR

@grosser grosser closed this Oct 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment