-
Notifications
You must be signed in to change notification settings - Fork 24
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
No TTL set on entity keys #3
Comments
+1 |
@mezis Has this now been fixed by redis-store/redis-store#209? |
@radar it has indeed. Sorry I didn't ping back to this issue! |
Well, to be specific, when The metastore still doesn't support native TTLs. |
we were experiencing the problem with the metastore keys. |
Ok. Is another patch needed here? |
@radar there would, but the issue has to be solved in rtomayko/rack-cache#92 has a proposed fix but it's been open for a long while. |
Our Redis instance filled up with keys because there's no TTL set on the entity keys. I see that Rack::Cache doesn't provide TTL to the MetaStore#write method, but I'm wondering if there's a suggested workaround or fix that could be done to prevent an unbounded number of keys. I'd be willing to do the work if there's an acceptable solution.
The text was updated successfully, but these errors were encountered: