use common retry policy builder function

Author: symbiogenesis

Projects/Dotmim.Sync.Core/Orchestrators/ApplyChanges/BaseOrchestrator.ApplyChanges.cs

@@ -242,15 +242,8 @@ internal virtual async Task<Exception> InternalApplyTableChangesAsync(ScopeInfo
 
                 this.Logger.LogInformation("[InternalApplyTableChangesAsync]. Directory name {DirectoryName}. BatchParts count {BatchPartsInfoCount}", message.Changes.DirectoryName, message.Changes.BatchPartsInfo.Count);
 
-                // If we have a transient error happening, and we are rerunning the tranaction,
-                // raising an interceptor
-                var onRetry = new Func<Exception, int, TimeSpan, object, Task>((ex, cpt, ts, arg) =>
-                    this.InterceptAsync(new TransientErrorOccuredArgs(context, connection, ex, cpt, ts), progress, cancellationToken).AsTask());
-
                 // Defining my retry policy
-                var retryPolicy = this.Options.TransactionMode != TransactionMode.AllOrNothing
-                    ? SyncPolicy.WaitAndRetryForever(retryAttempt => TimeSpan.FromMilliseconds(500 * retryAttempt), (ex, arg) => this.Provider.ShouldRetryOn(ex), onRetry)
-                    : SyncPolicy.WaitAndRetry(0, TimeSpan.Zero);
+                var retryPolicy = this.BuildSyncPolicy(context, connection, progress, cancellationToken);
 
                 var applyChangesPolicyResult = await retryPolicy.ExecuteAsync(
                     async () =>

Projects/Dotmim.Sync.Core/Orchestrators/ApplyChanges/LocalOrchestrator.ApplyChanges.cs

@@ -25,15 +25,8 @@ public partial class LocalOrchestrator : BaseOrchestrator
                               DbConnection connection = default, DbTransaction transaction = default,
                               IProgress<ProgressArgs> progress = null, CancellationToken cancellationToken = default)
         {
-            // If we have a transient error happening, and we are rerunning the tranaction,
-            // raising an interceptor
-            var onRetry = new Func<Exception, int, TimeSpan, object, Task>((ex, cpt, ts, arg) =>
-                this.InterceptAsync(new TransientErrorOccuredArgs(context, connection, ex, cpt, ts), progress, cancellationToken).AsTask());
-
             // Defining my retry policy
-            SyncPolicy retryPolicy = this.Options.TransactionMode == TransactionMode.AllOrNothing
-             ? retryPolicy = SyncPolicy.WaitAndRetryForever(retryAttempt => TimeSpan.FromMilliseconds(500 * retryAttempt), (ex, arg) => this.Provider.ShouldRetryOn(ex), onRetry)
-             : retryPolicy = SyncPolicy.WaitAndRetry(0, TimeSpan.Zero);
+            var retryPolicy = this.BuildSyncPolicy(context, connection, progress, cancellationToken);
 
             // Execute my OpenAsync in my policy context
             var applyChangesResult = await retryPolicy.ExecuteAsync<(SyncContext Context, ClientSyncChanges ClientSyncChange, ScopeInfoClient CScopeInfoClient)>(

Projects/Dotmim.Sync.Core/Orchestrators/ApplyChanges/RemoteOrchestrator.ApplyChanges.cs

@@ -52,15 +52,8 @@ public partial class RemoteOrchestrator : BaseOrchestrator
                 // Previous sync errors
                 BatchInfo lastSyncErrorsBatchInfo = null;
 
-                // If we have a transient error happening, and we are rerunning the tranaction,
-                // raising an interceptor
-                var onRetry = new Func<Exception, int, TimeSpan, object, Task>((ex, cpt, ts, arg) =>
-                    this.InterceptAsync(new TransientErrorOccuredArgs(context, connection, ex, cpt, ts), progress, cancellationToken).AsTask());
-
                 // Defining my retry policy
-                SyncPolicy retryPolicy = this.Options.TransactionMode == TransactionMode.AllOrNothing
-                 ? retryPolicy = SyncPolicy.WaitAndRetryForever(retryAttempt => TimeSpan.FromMilliseconds(500 * retryAttempt), (ex, arg) => this.Provider.ShouldRetryOn(ex), onRetry)
-                 : retryPolicy = SyncPolicy.WaitAndRetry(0, TimeSpan.Zero);
+                var retryPolicy = this.BuildSyncPolicy(context, connection, progress, cancellationToken);
 
                 await retryPolicy.ExecuteAsync(
                     async () =>

Projects/Dotmim.Sync.Core/Orchestrators/BaseOrchestrator.cs

@@ -445,5 +445,21 @@ internal SyncException GetSyncError(SyncContext context, Exception exception, st
                 throw this.GetSyncError(context, ex);
             }
         }
+
+        protected SyncPolicy BuildSyncPolicy(SyncContext context, DbConnection connection, IProgress<ProgressArgs> progress, CancellationToken cancellationToken, int retryIntervalMilliseconds = 500)
+        {
+            // If we have a transient error happening, and we are rerunning the transaction,
+            // raising an interceptor
+            var onRetry = new Func<Exception, int, TimeSpan, object, Task>((ex, cpt, ts, arg) =>
+                this.InterceptAsync(new TransientErrorOccuredArgs(context, connection, ex, cpt, ts), progress, cancellationToken).AsTask());
+
+            return this.Options.TransactionMode switch
+            {
+                TransactionMode.AllOrNothing => SyncPolicy.WaitAndRetry(0, TimeSpan.Zero),
+                TransactionMode.PerBatch => SyncPolicy.WaitAndRetryForever(retryAttempt => TimeSpan.FromMilliseconds(retryIntervalMilliseconds * retryAttempt), (ex, arg) => this.Provider.ShouldRetryOn(ex), onRetry),
+                TransactionMode.None => SyncPolicy.WaitAndRetryForever(retryAttempt => TimeSpan.FromMilliseconds(retryIntervalMilliseconds * retryAttempt), (ex, arg) => this.Provider.ShouldRetryOn(ex), onRetry),
+                _ => throw new NotImplementedException(),
+            };
+        }
     }
 }