Support from Acquia helps fund testing for Drupal Acquia logo

Comments

msti’s picture

I am also interested in this

PlayfulWolf’s picture

Mee too, as I am using BOA Aegir, which has Redis installed and enabled by default.
Also, with (my personal) history of memcached beeing much less stable than APC - Radioactivity could have more than 1 backend to store data: APC, memcached, redis, mongo...

Fidelix’s picture

Maybe if the maintainers provided a pluggable system to store information?
I would develop the Redis plugin in that case.

tcmug’s picture

Status: Active » Needs work

Actually the memcache storage class should serve as an example how to write a Redis/Whatnot storage.

It shouldn't be actually too hard to implement.

HyperGlide’s picture

PlayfulWolf’s picture

no news about redis/apc?

rodpal’s picture

This would be a great feature

wulff’s picture

AFAICT, Redis support is available in the latest development version of the module.

As mentioned above, adding APC-support should be fairly simple by using the included Memcache class as a template.

wulff’s picture

Issue summary: View changes
FileSize
2.96 KB

I've created a quick-and-dirty patch which shows a way to add APC support to the module.

tcmug’s picture

Dev version now has support for Redis, I will review the APC next

jfrederick’s picture

In addition to supporting Redis via the PhpRedis library, could the Predis library also be supported?

ZeiP’s picture

Attached is a patch to support both PhpRedis and Predis libraries for Redis access.

The patch also takes the Redis module settings as defaults for Radioactivity admin settings form and adds password authentication support for the connection with both libraries.

mvc’s picture

FileSize
9.36 KB

I've updated the above patch to set the Drupal root (since that was giving me watchdog errors in setPredisAutoload()) and to bootstrap the redis settings a little better. Works for us using PhpRedis.

alansaviolobo’s picture

Status: Needs work » Needs review
FileSize
8.7 KB

updated patch as a git diff.

mohit_aghera’s picture

Changing path of predis library in #14
sites/all/libraries/predis/lib/ to sites/all/libraries/predis/src/

SocialNicheGuru’s picture

patch -p1 < radioactivity-predis-support-1284228-15.patch
patching file includes/RadioactivityRedisIncidentStorage.inc
patching file radioactivity-admin-ui.inc
Hunk #1 succeeded at 144 (offset 11 lines).
Hunk #2 succeeded at 152 (offset 11 lines).
Hunk #3 succeeded at 175 (offset 11 lines).
Hunk #4 succeeded at 263 with fuzz 1 (offset 12 lines).
patching file radioactivity-bootstrap.inc
patch: **** malformed patch at line 168: @@ -53,6 +58,11 @@ function _radioactivity_light_initialization() {

mvc’s picture

Cleaned up patch from #15; now works with git apply.