-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature request - Redis support #11
Comments
This would be awesome |
<3 +1! |
This would be very awesome. Anyone had a chance to try this out? PR's encouraged! |
+1 I can't help with a PR, but with testing. |
Could we enable wp-redis support in wp-spider-cache? It is a high quality plugin. |
@JJJ I'd like to implement Redis support by wp-redis. Could you start me up where to modify the existing code to switch engine? |
Why does WP Spider Cache need Redis support when WP Redis already has Redis support (and comprehensive test coverage to boot)? |
I've tried wp-redis + wp-spider-cache/advanced-cache.php based on #21 (comment)
|
@szepeviktor I think what you're starting to do is a great approach:
Long term, I can imagine bundling our own shim to work with whatever WP Redis provides, and/or relying on it entirely and adapting it's approach for our own internals. |
I would like to have Redis support, because there is no decent page cache plugin at the moment that also supports Redis.
Would be nice to have support for:
(like this plugin has: https://nl.wordpress.org/plugins/redis-cache/)
If you have a good idea how to merge this in easily, it would be very nice. Otherwise I will have a look the in next month, if I can add Redis support and open a PR.
The text was updated successfully, but these errors were encountered: