Merge remote-tracking branch 'origin/master' into log-params-customize

vostok · Sep 3, 2024 · 9b035c7 · 9b035c7
2 parents 6fb0ba0 + 0dcd9de
commit 9b035c7
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 0.1.54 (27-08-2024):
+
+Don't validate stream content if retry is not needed
+
 ## 0.1.53 (06-08-2024):
 
 ConnectionTimeout for the last attempt in strategies now is "AlmostInfinity".

diff --git a/Vostok.ClusterClient.Core/Modules/RequestRetryModule.cs b/Vostok.ClusterClient.Core/Modules/RequestRetryModule.cs
@@ -34,15 +34,15 @@ public async Task<ClusterResult> ExecuteAsync(IRequestContext context, Func<IReq
                 if (context.Budget.HasExpired)
                     return result;
 
+                if (!retryPolicy.NeedToRetry(context.Request, context.Parameters, result.ReplicaResults))
+                    return result;
+
                 if (context.Request.ContainsAlreadyUsedStream() || context.Request.ContainsAlreadyUsedContent())
                 {
                     RequestMiscExtensions.LogRequestDataIsUsed();
                     return result;
                 }
 
-                if (!retryPolicy.NeedToRetry(context.Request, context.Parameters, result.ReplicaResults))
-                    return result;
-
                 attemptsUsed++;
                 var retryDelay = retryStrategy.GetRetryDelay(context, result, attemptsUsed);
                 if (!retryDelay.HasValue || retryDelay.Value >= context.Budget.Remaining)

diff --git a/Vostok.ClusterClient.Core/Vostok.ClusterClient.Core.csproj b/Vostok.ClusterClient.Core/Vostok.ClusterClient.Core.csproj
@@ -6,7 +6,7 @@
     <Configurations>Debug;Release</Configurations>
   </PropertyGroup>
   <PropertyGroup>
-    <VersionPrefix>0.1.54</VersionPrefix>
+    <VersionPrefix>0.1.55</VersionPrefix>
   </PropertyGroup>
   <PropertyGroup>
     <Title>Vostok.ClusterClient.Core</Title>