Release: Adds commits for 3.35.1 hotfix release (#3947)

* HttpTransport: Fixes HttpTimeoutPolicies to not accidentally suppress retries (#3944) * Fix HttpTimeoutPolicies to not accidentally suppress retries * Removing HttpTimeoutPolicy.MaxRetryTimeLimit altogether * SDK 3.35.1 : Adds version bump and changelog (#3945) * version bump * changelog * contract --------- Co-authored-by: Fabian Meiswinkel <fabian@meiswinkel.com>
Azure · Jun 27, 2023 · 947a5d0 · 947a5d0
1 parent b979061
commit 947a5d0
Show file tree

Hide file tree

Showing 10 changed files with 3,076 additions and 14 deletions.
diff --git a/Directory.Build.props b/Directory.Build.props
@@ -1,7 +1,7 @@
 <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <PropertyGroup>
- <ClientOfficialVersion>3.35.0</ClientOfficialVersion>
- <ClientPreviewVersion>3.35.0</ClientPreviewVersion>
+ <ClientOfficialVersion>3.35.1</ClientOfficialVersion>
+ <ClientPreviewVersion>3.35.1</ClientPreviewVersion>
  <ClientPreviewSuffixVersion>preview</ClientPreviewSuffixVersion>
  <DirectVersion>3.31.2</DirectVersion>
  <EncryptionOfficialVersion>2.0.2</EncryptionOfficialVersion>

diff --git a/Microsoft.Azure.Cosmos/contracts/API_3.35.1-preview.txt b/Microsoft.Azure.Cosmos/contracts/API_3.35.1-preview.txt
diff --git a/Microsoft.Azure.Cosmos/contracts/API_3.35.1.txt b/Microsoft.Azure.Cosmos/contracts/API_3.35.1.txt
diff --git a/Microsoft.Azure.Cosmos/src/HttpClient/CosmosHttpClientCore.cs b/Microsoft.Azure.Cosmos/src/HttpClient/CosmosHttpClientCore.cs
@@ -438,8 +438,7 @@ private static bool IsOutOfRetries(
  DateTime startDateTimeUtc,
  IEnumerator<(TimeSpan requestTimeout, TimeSpan delayForNextRequest)> timeoutEnumerator)
  {
- return (DateTime.UtcNow - startDateTimeUtc) > timeoutPolicy.MaximumRetryTimeLimit || // Maximum of time for all retries
- !timeoutEnumerator.MoveNext(); // No more retries are configured
+ return !timeoutEnumerator.MoveNext(); // No more retries are configured
  }
 
  private async Task<HttpResponseMessage> ExecuteHttpHelperAsync(

diff --git a/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicy.cs b/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicy.cs
@@ -11,7 +11,6 @@ namespace Microsoft.Azure.Cosmos
  internal abstract class HttpTimeoutPolicy
  {
  public abstract string TimeoutPolicyName { get; }
- public abstract TimeSpan MaximumRetryTimeLimit { get; }
  public abstract int TotalRetryCount { get; }
  public abstract IEnumerator<(TimeSpan requestTimeout, TimeSpan delayForNextRequest)> GetTimeoutEnumerator();
  public abstract bool IsSafeToRetry(HttpMethod httpMethod);

diff --git a/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicyControlPlaneRead.cs b/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicyControlPlaneRead.cs
@@ -25,8 +25,6 @@ private HttpTimeoutPolicyControlPlaneRead()
 
  public override string TimeoutPolicyName => HttpTimeoutPolicyControlPlaneRead.Name;
 
- public override TimeSpan MaximumRetryTimeLimit => CosmosHttpClient.GatewayRequestTimeout;
-
  public override int TotalRetryCount => this.TimeoutsAndDelays.Count;
 
  public override IEnumerator<(TimeSpan requestTimeout, TimeSpan delayForNextRequest)> GetTimeoutEnumerator()

diff --git a/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicyControlPlaneRetriableHotPath.cs b/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicyControlPlaneRetriableHotPath.cs
@@ -29,8 +29,6 @@ private HttpTimeoutPolicyControlPlaneRetriableHotPath(bool shouldThrow503OnTimeo
 
  public override string TimeoutPolicyName => HttpTimeoutPolicyControlPlaneRetriableHotPath.Name;
 
- public override TimeSpan MaximumRetryTimeLimit => CosmosHttpClient.GatewayRequestTimeout;
-
  public override int TotalRetryCount => this.TimeoutsAndDelays.Count;
 
  public override IEnumerator<(TimeSpan requestTimeout, TimeSpan delayForNextRequest)> GetTimeoutEnumerator()

diff --git a/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicyDefault.cs b/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicyDefault.cs
@@ -29,8 +29,6 @@ private HttpTimeoutPolicyDefault(bool shouldThrow503OnTimeout)
 
  public override string TimeoutPolicyName => HttpTimeoutPolicyDefault.Name;
 
- public override TimeSpan MaximumRetryTimeLimit => CosmosHttpClient.GatewayRequestTimeout;
-
  public override int TotalRetryCount => this.TimeoutsAndDelays.Count;
 
  public override IEnumerator<(TimeSpan requestTimeout, TimeSpan delayForNextRequest)> GetTimeoutEnumerator()

diff --git a/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicyNoRetry.cs b/Microsoft.Azure.Cosmos/src/HttpClient/HttpTimeoutPolicyNoRetry.cs
@@ -24,8 +24,6 @@ private HttpTimeoutPolicyNoRetry()
 
  public override string TimeoutPolicyName => HttpTimeoutPolicyNoRetry.Name;
 
- public override TimeSpan MaximumRetryTimeLimit => TimeSpan.Zero;
-
  public override int TotalRetryCount => 0;
 
  public override IEnumerator<(TimeSpan requestTimeout, TimeSpan delayForNextRequest)> GetTimeoutEnumerator()

diff --git a/changelog.md b/changelog.md
@@ -12,7 +12,14 @@ Preview features are treated as a separate branch and will not be included in th
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
-### <a name="3.34.0-preview"/> [3.35.0-preview](https://www.nuget.org/packages/Microsoft.Azure.Cosmos/3.35.0-preview) - 2023-06-19
+
+### <a name="3.35.1-preview"/> [3.35.1-preview](https://www.nuget.org/packages/Microsoft.Azure.Cosmos/3.35.1-preview) - 2023-06-27
+### <a name="3.35.1"/> [3.35.1](https://www.nuget.org/packages/Microsoft.Azure.Cosmos/3.35.1) - 2023-06-27
+
+#### Fixed
+- [3944](https://github.com/Azure/azure-cosmos-dotnet-v3/pull/3944) Availability: Fixes HttpTimeoutPolicies to not accidentally suppress retries
+
+### <a name="3.35.0-preview"/> [3.35.0-preview](https://www.nuget.org/packages/Microsoft.Azure.Cosmos/3.35.0-preview) - 2023-06-19
 
 ### Added
 - [3836](https://github.com/Azure/azure-cosmos-dotnet-v3/pull/3836) Integrated cache: Adds BypassIntegratedCache to DedicatedGatewayRequestOptions