Add cache_clear operation
#200
Conversation
src/stores/redis.rs
Outdated
| // https://redis.io/commands/flushdb/ | ||
| let _: () = redis::cmd("FLUSHDB").query(&mut *conn)?; |
There was a problem hiding this comment.
Wouldn't this delete someone's entire redis db? I think a scan + delete would need to be done instead, using the self.namespace + self.prefix in the scan
There was a problem hiding this comment.
Oh I just assumed the Redis DB was already unique per cache. I can fix this.
There was a problem hiding this comment.
I took a shot at it but needing two connections seems weird. scan_match needs &mut self, though, so I can't do anything with the deserialized values without opening another connection or collecting them all into memory first.
https://docs.rs/redis/latest/redis/trait.Commands.html#method.scan_match
There was a problem hiding this comment.
Any plans to merge this?
There was a problem hiding this comment.
Took a while, but I reimplemented this with the scan + delete strategy. Should be ready now?
|
Hi all, is there any development regarding this feature? Was it possibly added in some other PR? I am looking for a way to clear the redis cache. |
9999years
force-pushed
the
cache-clear
branch
from
228daae to
6cdcd56
Compare
9999years
force-pushed
the
cache-clear
branch
from
6cdcd56 to
7d2342b
Compare
|
@LucaCappelletti94 I just fixed up the merge conflicts and fixed up the async Redis |
Closes #197.