Skip to content

Commit 7e3bc8b

Browse files
author
Mitch Hagstrand
committed
Removed memcached.sess_consistent_hashing option.
1. Removed memcached.sess_consistent_hashing which is a lesser version of memcached.sess_consistent_hash 2. Added default value for memcached.sess_consistent_hash
1 parent 8230dc1 commit 7e3bc8b

File tree

4 files changed

+4
-14
lines changed

4 files changed

+4
-14
lines changed

php_memcached.c

+1-2
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,6 @@ PHP_INI_BEGIN()
292292

293293
STD_PHP_INI_ENTRY("memcached.sess_number_of_replicas", "0", PHP_INI_ALL, OnUpdateLongGEZero, sess_number_of_replicas, zend_php_memcached_globals, php_memcached_globals)
294294
STD_PHP_INI_ENTRY("memcached.sess_randomize_replica_read", "0", PHP_INI_ALL, OnUpdateBool, sess_randomize_replica_read, zend_php_memcached_globals, php_memcached_globals)
295-
STD_PHP_INI_ENTRY("memcached.sess_consistent_hashing", "0", PHP_INI_ALL, OnUpdateBool, sess_consistent_hashing_enabled, zend_php_memcached_globals, php_memcached_globals)
296295
STD_PHP_INI_ENTRY("memcached.sess_remove_failed", "0", PHP_INI_ALL, OnUpdateBool, sess_remove_failed_enabled, zend_php_memcached_globals, php_memcached_globals)
297296
#endif
298297
STD_PHP_INI_ENTRY("memcached.compression_type", "fastlz", PHP_INI_ALL, OnUpdateCompressionType, compression_type, zend_php_memcached_globals, php_memcached_globals)
@@ -3042,7 +3041,7 @@ static void php_memc_init_globals(zend_php_memcached_globals *php_memcached_glob
30423041
#ifdef HAVE_MEMCACHED_SESSION
30433042
MEMC_G(sess_locking_enabled) = 1;
30443043
MEMC_G(sess_binary_enabled) = 1;
3045-
MEMC_G(sess_consistent_hashing_enabled) = 0;
3044+
MEMC_G(sess_consistent_hash_enabled) = 0;
30463045
MEMC_G(sess_number_of_replicas) = 0;
30473046
MEMC_G(sess_remove_failed_enabled) = 0;
30483047
MEMC_G(sess_prefix) = NULL;

php_memcached.h

+2-3
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,8 @@ ZEND_BEGIN_MODULE_GLOBALS(php_memcached)
7070
int sess_number_of_replicas;
7171
zend_bool sess_randomize_replica_read;
7272
zend_bool sess_remove_failed_enabled;
73-
zend_bool sess_consistent_hashing_enabled;
73+
zend_bool sess_consistent_hash_enabled;
74+
zend_bool sess_binary_enabled;
7475
#endif
7576
char *serializer_name;
7677
enum memcached_serializer serializer;
@@ -83,8 +84,6 @@ ZEND_BEGIN_MODULE_GLOBALS(php_memcached)
8384
#if HAVE_MEMCACHED_SASL
8485
bool use_sasl;
8586
#endif
86-
zend_bool sess_consistent_hash_enabled;
87-
zend_bool sess_binary_enabled;
8887
ZEND_END_MODULE_GLOBALS(php_memcached)
8988

9089
PHP_MEMCACHED_API zend_class_entry *php_memc_get_ce(void);

php_memcached_session.c

-7
Original file line numberDiff line numberDiff line change
@@ -228,13 +228,6 @@ PS_OPEN_FUNC(memcached)
228228
}
229229
}
230230

231-
if (MEMC_G(sess_consistent_hashing_enabled)) {
232-
if (memcached_behavior_set(memc_sess->memc_sess, MEMCACHED_BEHAVIOR_KETAMA, (uint64_t) 1) == MEMCACHED_FAILURE) {
233-
php_error_docref(NULL TSRMLS_CC, E_WARNING, "failed to set memcached consistent hashing");
234-
return FAILURE;
235-
}
236-
}
237-
238231
/* Allow libmemcached remove failed servers */
239232
if (MEMC_G(sess_remove_failed_enabled)) {
240233
if (memcached_behavior_set(memc_sess->memc_sess, MEMCACHED_BEHAVIOR_REMOVE_FAILED_SERVERS, (uint64_t) 1) == MEMCACHED_FAILURE) {

tests/experimental/moduleinfo.phpt

+1-2
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,9 @@ memcached.compression_type => %s => %s
2525
memcached.serializer => %s => %s
2626
memcached.sess_binary => %d => %d
2727
memcached.sess_consistent_hash => %d => %d
28-
memcached.sess_consistent_hashing => %d => %d
2928
memcached.sess_lock_wait => %d => %d
3029
memcached.sess_locking => %d => %d
3130
memcached.sess_number_of_replicas => %d => %d
3231
memcached.sess_prefix => %s => %s
3332
memcached.sess_randomize_replica_read => %d => %d
34-
memcached.sess_remove_failed => %d => %d
33+
memcached.sess_remove_failed => %d => %d

0 commit comments

Comments
 (0)