For an application I have been working on we have noticed some limitations associated with using PhpRedis. When it comes to stack tracing and performance testing I have found it impossible to debug what happens after Drupal hands a request off to PhpRedis. Since Predis is a PHP library, it should be easier to trace what it is doing and how long it is taking to perform a specific task.

I have made a first attempt at a Predis implementation here in the patch file. I have tried to limit my influence on the module structure so that Predis support is something that is just "dropped in" and does not affect any sites that are currently happily using PhpRedis.

Feedback welcome :)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

maniosullivan created an issue. See original summary.

maniosullivan’s picture

maniosullivan’s picture

I noticed that I had not implemented pipelining correctly for Predis. Updated patch attached.

maniosullivan’s picture

Assigned: maniosullivan » Unassigned
maniosullivan’s picture

I have added some basic support for a master slave Redis replication using Predis.

maniosullivan’s picture

Status: Active » Needs review
maniosullivan’s picture

Updated some terminology in the README.

Berdir credited Berdir.

Berdir’s picture

Status: Needs review » Fixed

This has been finished on https://github.com/md-systems/redis/pull/25, merged and committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.