@jetit/publisher 1.1.1 → 1.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jetit/publisher",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "type": "commonjs",
   "dependencies": {
     "@jetit/id": "0.0.11",

package/src/lib/redis/scheduler.js

@@ -51,7 +51,7 @@ class ScheduledProcessor {
                  * Instead of using the publish method directly, the entire logic is
                  * copy pasted to reduce the case of failure.
                  */
-                const transaction = this.redisPublisher.multi();
+                const transaction = this.redisPublisher.multi({ pipeline: true });
                 eventData.eventId = (0, id_1.generateID)('HEX', 'FF');
                 transaction.zrem('se', eventString);
                 const consumerGroups = yield (0, groups_1.getAllConsumerGroups)(eventData.eventName, this.redisPublisher);

package/src/lib/redis/streams.js

@@ -110,7 +110,7 @@ class Streams {
             data.eventId = (0, id_1.generateID)('HEX', 'FF');
             if (!data.createdAt)
                 data.createdAt = Date.now();
-            const transaction = this.redisPublisher.multi();
+            const transaction = this.redisPublisher.multi({ pipeline: true });
             const consumerGroups = yield (0, groups_1.getAllConsumerGroups)(data.eventName, this.redisPublisher);
             if (consumerGroups.length > 0) {
                 console.log(`Publishing event ${data.eventName} to consumer groups: ${consumerGroups.join(', ')}`);
@@ -236,7 +236,7 @@ class Streams {
                                 bs.next(eventData);
                                 const pmKey = `pm:${this.consumerGroupName}:${streamName}`;
                                 const currentTime = Date.now();
-                                const transaction = this.redisGroups.multi();
+                                const transaction = this.redisGroups.multi({ pipeline: true });
                                 transaction.zadd(pmKey, currentTime, messageId);
                                 transaction.xack(streamName, this.consumerGroupName, id);
                                 transaction.zadd(`ack:${streamName}`, Date.now(), id);
@@ -275,8 +275,10 @@ class Streams {
             const result = yield this.redisGroups.xreadgroup('GROUP', this.consumerGroupName, this.instanceId, 'STREAMS', streamName, '>');
             if (result) {
                 const [, streamMessages] = result[0];
+                if (!streamMessages)
+                    return;
                 console.log(`Unprocessed events: ${streamMessages.length}`);
-                const transaction = this.redisGroups.multi();
+                const transaction = this.redisGroups.multi({ pipeline: true });
                 for (const [id, data] of streamMessages) {
                     const eventData = JSON.parse(data[1]);
                     // Republishing the events
@@ -312,7 +314,7 @@ class Streams {
             const [, minId, maxId, consumers] = pendingMessages;
             if (!consumers || consumers.length === 0)
                 return;
-            const transaction = this.redisGroups.multi();
+            const transaction = this.redisGroups.multi({ pipeline: true });
             for (const [consumer, pendingCount] of consumers) {
                 if (parseInt(pendingCount) > 0) {
                     const pending = (yield this.redisGroups.xpending(streamName, this.consumerGroupName, minId, maxId, Number(pendingCount), consumer));
@@ -387,7 +389,7 @@ class Streams {
             const cleanupThreshold = Date.now() - interval;
             const acknowledgedMessages = yield this.redisGroups.zrangebyscore(`ack:${streamName}`, '-inf', cleanupThreshold);
             if (acknowledgedMessages && acknowledgedMessages.length > 0) {
-                const transaction = this.redisGroups.multi();
+                const transaction = this.redisGroups.multi({ pipeline: true });
                 // Remove acknowledged messages from the stream
                 for (const messageId of acknowledgedMessages) {
                     transaction.xdel(streamName, messageId);