-
Notifications
You must be signed in to change notification settings - Fork 60
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
Caching Implementation Inconsistencies #179
Comments
@nnordrum/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot! |
I realize |
@nnordrum Would an extra constructor with |
@andreoss what's up with this ticket? |
@yegor256 I would go with extra parameter to |
@andreoss yes, it's fine |
There currently (that I'm aware of) isn't a way to externalize the cache to a Reddis/Cassandra/etc.
There are a number of caching wires that are unable to do the job:
AbstractHeaderBasedCachingWire.java
: cache is a transientMap
that isn't user replaceableCachingWire.java
: uses guavaMap
, so probably the closest, but not easy/documented how to do itFcWire.java
: cache root is a directory, andFcCache
is final and not an InterfaceAll of these are close, but no cigar.
What I'm specifically trying to do is create a Wire (
GitHubWire.java
) that supports a pluggable cache and supportsETag
.AbstractHeaderBasedCachingWire.java
would be perfect if I could just swap out the caching implementation...The text was updated successfully, but these errors were encountered: