From 2570eeb711ffb969d5532a8fd5d90b4d05af0792 Mon Sep 17 00:00:00 2001 From: antirez Date: Fri, 18 Jul 2014 10:09:51 +0200 Subject: [PATCH] tryObjectEncoding(): use shared objects with maxmemory and non-LRU policy. In order to make sure every object has its own private LRU counter, when maxmemory is enabled tryObjectEncoding() does not use the pool of shared integers. However when the policy is not LRU-based, it does not make sense to do so, and it is much better to save memory using shared integers. --- src/object.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/object.c b/src/object.c index 2cf9e04f..6b8e4247 100644 --- a/src/object.c +++ b/src/object.c @@ -373,7 +373,9 @@ robj *tryObjectEncoding(robj *o) { * Note that we avoid using shared integers when maxmemory is used * because every object needs to have a private LRU field for the LRU * algorithm to work well. */ - if (server.maxmemory == 0 && + if ((server.maxmemory == 0 || + (server.maxmemory_policy != REDIS_MAXMEMORY_VOLATILE_LRU && + server.maxmemory_policy != REDIS_MAXMEMORY_ALLKEYS_LRU)) && value >= 0 && value < REDIS_SHARED_INTEGERS) {