fix(redis): fix race condition that can result in message duplication (…

…#48)
spinnaker · Sep 21, 2018 · 2d96768 · 2d96768
1 parent e9a749e
commit 2d96768
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/keiko-redis/src/main/kotlin/com/netflix/spinnaker/q/redis/RedisQueue.kt b/keiko-redis/src/main/kotlin/com/netflix/spinnaker/q/redis/RedisQueue.kt
@@ -130,7 +130,7 @@ class RedisQueue(
         if (fingerprint != null) {
           log.info("Re-prioritizing message as an identical one is already on the queue: " +
             "$fingerprint, message: $message")
-          redis.zadd(queueKey, score(delay), fingerprint)
+          redis.zadd(queueKey, score(delay), fingerprint, zAddParams().xx())
           fire(MessageDuplicate(message))
         } else {
           redis.queueMessage(message, delay)