[mms] Predis client should only use clustering if necessary.

horde · Jan 3, 2014 · 8680d18 · 8680d18
1 parent 9597ede
commit 8680d18
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/framework/Core/lib/Horde/Core/Factory/HashTable.php b/framework/Core/lib/Horde/Core/Factory/HashTable.php
@@ -70,6 +70,11 @@ public function create(Horde_Injector $injector)
                 }
             }
 
+            /* No need to use complex clustering if not needed. */
+            if (count($redis_params) === 1) {
+                $redis_params = reset($redis_params);
+            }
+
             return new Horde_HashTable_Predis(array(
                 'predis' => new Predis\Client($redis_params)
             ));

diff --git a/framework/Core/package.xml b/framework/Core/package.xml
@@ -39,6 +39,7 @@
  </stability>
  <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
  <notes>
+* [mms] Predis client should only use clustering if necessary.
 * [jan] Fix retrieving value lists from Horde_Session.
 * [mjr] Use webnotifications.js to normalize various browsers to the standard Notification API.
 * [mjr] Fix desktop style notifications in Ajax applications (Bug #12835).
@@ -3264,6 +3265,7 @@
    <date>2013-12-07</date>
    <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
    <notes>
+* [mms] Predis client should only use clustering if necessary.
 * [jan] Fix retrieving value lists from Horde_Session.
 * [mjr] Use webnotifications.js to normalize various browsers to the standard Notification API.
 * [mjr] Fix desktop style notifications in Ajax applications (Bug #12835).