Put more details in the comment introduced by #4601.

1 files changed, 8 insertions, 3 deletions

diff --git a/src/lazyfree.c b/src/lazyfree.c
index cfab6a31a..f1de0c898 100644
--- a/src/lazyfree.c
+++ b/src/lazyfree.c
@@ -64,9 +64,14 @@ int dbAsyncDelete(redisDb *db, robj *key) {
         robj *val = dictGetVal(de);
         size_t free_effort = lazyfreeGetFreeEffort(val);
 
-        /* If releasing the object is too much work and the refcount
-         * is 1, that means the object really needs to be freed,
-         * let's put it into the lazy free list. */
+        /* If releasing the object is too much work, do it in the background
+         * by adding the object to the lazy free list.
+         * Note that if the object is shared, to reclaim it now it is not
+         * possible. This rarely happens, however sometimes the implementation
+         * of parts of the Redis core may call incrRefCount() to protect
+         * objects, and then call dbDelete(). In this case we'll fall
+         * through and reach the dictFreeUnlinkedEntry() call, that will be
+         * equivalent to just calling decrRefCount(). */
         if (free_effort > LAZYFREE_THRESHOLD && val->refcount == 1) {
             atomicIncr(lazyfree_objects,1);
             bioCreateBackgroundJob(BIO_LAZY_FREE,val,NULL,NULL);