Move one messager after the other from list to stream

alexander-schranz · alexander-schranz · commit 7407abd8e5ef · 2019-07-09T10:43:52.000+02:00
diff --git a/src/Symfony/Component/Messenger/Transport/RedisExt/Connection.php b/src/Symfony/Component/Messenger/Transport/RedisExt/Connection.php
@@ -58,9 +58,9 @@ public function __construct(array $configuration, array $connectionCredentials =
         }
 
         $this->stream = $configuration['stream'] ?? self::DEFAULT_OPTIONS['stream'];
-        $this->queue = $this->stream.'_queue';
         $this->group = $configuration['group'] ?? self::DEFAULT_OPTIONS['group'];
         $this->consumer = $configuration['consumer'] ?? self::DEFAULT_OPTIONS['consumer'];
+        $this->queue = $this->stream.'__queue';
         $this->autoSetup = $configuration['auto_setup'] ?? self::DEFAULT_OPTIONS['auto_setup'];
         $this->maxEntries = $configuration['stream_max_entries'] ?? self::DEFAULT_OPTIONS['stream_max_entries'];
     }
@@ -117,12 +117,14 @@ public function get(): ?array
         $queuedMessageCount = $this->connection->zcount($this->queue, 0, time());
 
         if ($queuedMessageCount) {
-            foreach ($this->connection->zpopmin($this->queue, $queuedMessageCount) as $queuedMessage => $time) {
-                $queuedMessage = json_encode($queuedMessage, true);
-                // if a futured placed message is actually popped because of a race condition with
-                // another running message consumer, the message is readded to the queue by add function
-                // else its just added stream and will be available for all stream consumers
-                $this->add($queuedMessage['body'], $queuedMessage['headers'], (time() - $time) * 1000);
+            for ($i = 0; $i < $queuedMessageCount; ++$i) {
+                foreach ($this->connection->zpopmin($this->queue, 1) as $queuedMessage => $time) {
+                    $queuedMessage = json_encode($queuedMessage, true);
+                    // if a futured placed message is actually popped because of a race condition with
+                    // another running message consumer, the message is readded to the queue by add function
+                    // else its just added stream and will be available for all stream consumers
+                    $this->add($queuedMessage['body'], $queuedMessage['headers'], (time() - $time) * 1000);
+                }
             }
         }
 
