Merge pull request #533 from Particular/port-disable-delayed-delivery

Allow disabling delayed delivery
Particular · Nov 21, 2019 · 6753dd9 · 6753dd9
2 parents cb787c6 + fe23ab5
commit 6753dd9
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 18 deletions.
diff --git a/src/NServiceBus.SqlServer/Configuration/SettingsKeys.cs b/src/NServiceBus.SqlServer/Configuration/SettingsKeys.cs
@@ -23,5 +23,10 @@ class SettingsKeys
         /// For testing the migration process only
         /// </summary>
         public const string DisableNativePubSub = "SqlServer.DisableNativePubSub";
+
+        /// <summary>
+        /// For endpoints that only consume messages e.g. ServiceControl error
+        /// </summary>
+        public const string DisableDelayedDelivery = "SqlServer.DisableDelayedDelivery";
     }
 }
diff --git a/src/NServiceBus.SqlServer/Receiving/QueueCreator.cs b/src/NServiceBus.SqlServer/Receiving/QueueCreator.cs
@@ -31,8 +31,11 @@ public async Task CreateQueueIfNecessary(QueueBindings queueBindings, string ide
                 {
                     await CreateQueue(SqlConstants.CreateQueueText, addressTranslator.Parse(sendingAddress), connection, transaction, createMessageBodyColumn).ConfigureAwait(false);
                 }
-
-                await CreateQueue(SqlConstants.CreateDelayedMessageStoreText, delayedQueueAddress, connection, transaction, createMessageBodyColumn).ConfigureAwait(false);
+
+                if (delayedQueueAddress != null)
+                {
+                    await CreateQueue(SqlConstants.CreateDelayedMessageStoreText, delayedQueueAddress, connection, transaction, createMessageBodyColumn).ConfigureAwait(false);
+                }
                 transaction.Commit();
             }
         }

diff --git a/src/NServiceBus.SqlServer/SqlServerTransportInfrastructure.cs b/src/NServiceBus.SqlServer/SqlServerTransportInfrastructure.cs
@@ -145,23 +145,27 @@ public override TransportReceiveInfrastructure ConfigureReceiveInfrastructure()
             Func<string, TableBasedQueue> queueFactory = queueName => new TableBasedQueue(addressTranslator.Parse(queueName).QualifiedTableName, queueName);
 
             //Create delayed delivery infrastructure
-            var delayedDeliverySettings = settings.GetOrDefault<DelayedDeliverySettings>();
-            settings.AddStartupDiagnosticsSection("NServiceBus.Transport.SqlServer.DelayedDelivery", new
+            CanonicalQueueAddress delayedQueueCanonicalAddress = null;
+            if (false == settings.GetOrDefault<bool>(SettingsKeys.DisableDelayedDelivery))
             {
-                Native = true,
-                delayedDeliverySettings.Suffix,
-                delayedDeliverySettings.Interval,
-                BatchSize = delayedDeliverySettings.MatureBatchSize,
-                TimoutManager = delayedDeliverySettings.EnableMigrationMode ? "enabled" : "disabled"
-            });
-
-            var delayedQueueCanonicalAddress = GetDelayedTableAddress(delayedDeliverySettings);
-            var inputQueueTable = addressTranslator.Parse(ToTransportAddress(logicalAddress())).QualifiedTableName;
-            var delayedMessageTable = new DelayedMessageTable(delayedQueueCanonicalAddress.QualifiedTableName, inputQueueTable);
-
-            //Allows dispatcher to store messages in the delayed store
-            delayedMessageStore = delayedMessageTable;
-            dueDelayedMessageProcessor = new DueDelayedMessageProcessor(delayedMessageTable, connectionFactory, delayedDeliverySettings.Interval, delayedDeliverySettings.MatureBatchSize);
+                var delayedDeliverySettings = settings.GetOrDefault<DelayedDeliverySettings>();
+                settings.AddStartupDiagnosticsSection("NServiceBus.Transport.SqlServer.DelayedDelivery", new
+                {
+                    Native = true,
+                    delayedDeliverySettings.Suffix,
+                    delayedDeliverySettings.Interval,
+                    BatchSize = delayedDeliverySettings.MatureBatchSize,
+                    TimoutManager = delayedDeliverySettings.EnableMigrationMode ? "enabled" : "disabled"
+                });
+
+                delayedQueueCanonicalAddress = GetDelayedTableAddress(delayedDeliverySettings);
+                var inputQueueTable = addressTranslator.Parse(ToTransportAddress(logicalAddress())).QualifiedTableName;
+                var delayedMessageTable = new DelayedMessageTable(delayedQueueCanonicalAddress.QualifiedTableName, inputQueueTable);
+
+                //Allows dispatcher to store messages in the delayed store
+                delayedMessageStore = delayedMessageTable;
+                dueDelayedMessageProcessor = new DueDelayedMessageProcessor(delayedMessageTable, connectionFactory, delayedDeliverySettings.Interval, delayedDeliverySettings.MatureBatchSize);
+            }
 
             return new TransportReceiveInfrastructure(
                 () => new MessagePump(receiveStrategyFactory, queueFactory, queuePurger, expiredMessagesPurger, queuePeeker, schemaVerification, waitTimeCircuitBreaker),