From: martinlanghoff <martinlanghoff>
Date: Wed, 27 Dec 2006 22:45:13 +0000 (+0000)
Subject: init_memcached(): only use pconnect if configured to do so
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=9c967c33fb78624d0f360b4a3cac684ea09a45a0;p=moodle.git

init_memcached(): only use pconnect if configured to do so

Make pconnections optional - only use pconnect if $CFG->memcachedpconn is
set. In current versions of the memcache client in PECL (v2.0.1),
memcache_pconnect() is buggy and will segfault if reusing a pconnection
to a server that has gone away or restarted.

And hope for a more stable client library in PECL ;-)
---

diff --git a/lib/setuplib.php b/lib/setuplib.php
index d8e19a3bb1..2fe294a30a 100644
--- a/lib/setuplib.php
+++ b/lib/setuplib.php
@@ -219,9 +219,11 @@ function init_memcached() {
                                                
     $hosts = split(',', $CFG->memcachedhosts);
     $MCACHE = new Memcache;
-    if (count($hosts) === 1) {
+    if (count($hosts) === 1 && !empty($CFG->memcachedpconn)) {
         // the faster pconnect is only available
         // for single-server setups
+        // NOTE: PHP-PECL client is buggy and pconnect() 
+        // will segfault if the server is unavailable
         $MCACHE->pconnect($hosts[0]);
     } else {
         // multi-host setup will share key space