Updates the throttle timeout to be part of the options

Shazwazza · Shazwazza · commit eea90e78c3f5 · 2014-12-23T15:10:27.000+11:00
diff --git a/src/Umbraco.Core/Sync/DatabaseServerMessenger.cs b/src/Umbraco.Core/Sync/DatabaseServerMessenger.cs
@@ -48,8 +48,6 @@ public DatabaseServerMessenger(ApplicationContext appContext, bool enableDistCal
         private readonly object _lock = new object();
         private int _lastId = -1;
         private volatile bool _syncing = false;
-        //this ensures that only one thread can possibly check for sync operations every 5 seconds
-        private const int SyncTimeFrameSeconds = 5;
         private long _lastUtcTicks;
 
         /// <summary>
@@ -118,7 +116,7 @@ internal void Sync()
         {
 
             //don't process, this is not in the timeframe - we don't want to check the db every request, only once if it's been at least 5 seconds.
-            if (TimeSpan.FromTicks(DateTime.UtcNow.Ticks).TotalSeconds - TimeSpan.FromTicks(_lastUtcTicks).TotalSeconds <= SyncTimeFrameSeconds)
+            if (TimeSpan.FromTicks(DateTime.UtcNow.Ticks).TotalSeconds - TimeSpan.FromTicks(_lastUtcTicks).TotalSeconds <= _options.ThrottleSeconds)
             {
                 //NOTE: Removed logging as it will just keep showing this and people will wonder why.
                 //LogHelper.Debug<DatabaseServerMessenger>("Skipping distributed sync, not in timeframe");
diff --git a/src/Umbraco.Core/Sync/DatabaseServerMessengerOptions.cs b/src/Umbraco.Core/Sync/DatabaseServerMessengerOptions.cs
@@ -5,8 +5,15 @@ internal class DatabaseServerMessengerOptions
         public DatabaseServerMessengerOptions()
         {
             DaysToRetainInstructionRecords = 100;
+            ThrottleSeconds = 5;
         }
 
         public int DaysToRetainInstructionRecords { get; set; }
+
+        /// <summary>
+        /// The number of seconds to wait between previous sync operations - this ensures that sync operations
+        /// are not performed too often.
+        /// </summary>
+        public int ThrottleSeconds { get; set; }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -5,8 +5,15 @@ internal class DatabaseServerMessengerOptions`
`5`	`5`	`public DatabaseServerMessengerOptions()`
`6`	`6`	`{`
`7`	`7`	`DaysToRetainInstructionRecords = 100;`
	`8`	`+ ThrottleSeconds = 5;`
`8`	`9`	`}`
`9`	`10`
`10`	`11`	`public int DaysToRetainInstructionRecords { get; set; }`
	`12`	`+`
	`13`	`+ /// <summary>`
	`14`	`+ /// The number of seconds to wait between previous sync operations - this ensures that sync operations`
	`15`	`+ /// are not performed too often.`
	`16`	`+ /// </summary>`
	`17`	`+ public int ThrottleSeconds { get; set; }`
`11`	`18`	`}`
`12`	`19`	`}`