Do not instantiate multiple Random instances (#996)

dotnet · Mar 19, 2021 · 0cd0971 · 0cd0971
1 parent 9ebafbd
commit 0cd0971
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/...ft.Data.SqlClient/src/Microsoft/Data/SqlClient/Reliability/SqlRetryIntervalEnumerators.cs b/...ft.Data.SqlClient/src/Microsoft/Data/SqlClient/Reliability/SqlRetryIntervalEnumerators.cs
@@ -82,6 +82,7 @@ internal class SqlFixedIntervalEnumerator : SqlRetryIntervalBaseEnumerator
     {
         private readonly int maxRandom;
         private readonly int minRandom;
+        private readonly Random random = new Random();
 
         public SqlFixedIntervalEnumerator(TimeSpan gapTimeInterval, TimeSpan maxTimeInterval, TimeSpan minTimeInterval)
             : base(gapTimeInterval, maxTimeInterval, minTimeInterval)
@@ -94,7 +95,6 @@ public SqlFixedIntervalEnumerator(TimeSpan gapTimeInterval, TimeSpan maxTimeInte
 
         protected override TimeSpan GetNextInterval()
         {
-            var random = new Random();
             Current = TimeSpan.FromMilliseconds(random.Next(minRandom, maxRandom));
             return Current;
         }