From c88554074987a461600217411cd6cd88b96a5331 Mon Sep 17 00:00:00 2001 From: Jon Chambers Date: Mon, 5 Oct 2020 10:29:00 -0400 Subject: [PATCH] Check that the return of ZRANGEBYSCORE isn't an empty list. --- service/src/main/resources/lua/repair_queue_metadata.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/service/src/main/resources/lua/repair_queue_metadata.lua b/service/src/main/resources/lua/repair_queue_metadata.lua index 2b3a7e6ae..29d850427 100644 --- a/service/src/main/resources/lua/repair_queue_metadata.lua +++ b/service/src/main/resources/lua/repair_queue_metadata.lua @@ -6,7 +6,7 @@ local lastMessageId = tonumber(redis.call("ZRANGE", queueKey, -1, -1, "WITHSCOR if firstMessageId and lastMessageId then for messageId = firstMessageId,lastMessageId do - if redis.call("ZRANGEBYSCORE", queueKey, messageId, messageId) then + if next(redis.call("ZRANGEBYSCORE", queueKey, messageId, messageId)) then -- This message actually exists, and its GUID may be pointing to the wrong ID local guid = redis.call("HGET", queueMetadataKey, messageId .. "guid") redis.call("HSET", queueMetadataKey, guid, messageId)