Permalink
Browse files

MDL-41410 Caching: move comment to the appropriate place.

MDL-38162 fixes the issue about testing the connection to the memcached
daemon, which means testing once the connection to the pool of servers
and not to each of the servers while being added to the pool.
It misses to move the comment to that code too.
  • Loading branch information...
1 parent abb7671 commit 91b9fb37491492b546c681541d9fd8dae5ecce86 @scara scara committed Aug 25, 2013
Showing with 2 additions and 1 deletion.
  1. +1 −1 cache/stores/memcache/lib.php
  2. +1 −0 cache/stores/memcached/lib.php
@@ -132,8 +132,8 @@ public function __construct($name, array $configuration = array()) {
$this->connection = new Memcache;
foreach ($this->servers as $server) {
$this->connection->addServer($server[0], $server[1], true, $server[2]);
- // Test the connection to this server.
}
+ // Test the connection to the pool of servers.
$this->isready = @$this->connection->set($this->parse_key('ping'), 'ping', MEMCACHE_COMPRESSED, 1);
}
@@ -141,6 +141,7 @@ public function __construct($name, array $configuration = array()) {
}
$this->connection->addServers($this->servers);
}
+ // Test the connection to the pool of servers.
$this->isready = @$this->connection->set("ping", 'ping', 1);
}

0 comments on commit 91b9fb3

Please sign in to comment.