Skip to content

[DO NOT MERGE]: Leverage x-ms-cosmos-hub-region-processing-only for 404 Read Session Not Available cross-region retry scenarios. #47631

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
jeet1995 wants to merge 18 commits into
base: main
Choose a base branch
from
Draft

[DO NOT MERGE]: Leverage x-ms-cosmos-hub-region-processing-only for 404 Read Session Not Available cross-region retry scenarios. #47631

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
f2947d8
Adding `x-ms-cosmos-hub-region-processing-only` header.
jeet1995 Jan 5, 2026
aa521c5
Merge branch 'main' of https://github.com/jeet1995/azure-sdk-for-java…
jeet1995 Jan 7, 2026
a55d645
Adding `x-ms-cosmos-hub-region-processing-only` header.
jeet1995 Jan 8, 2026
d14a25c
Ensuring account-level region cycle through when `x-ms-cosmos-hub-reg…
jeet1995 Jan 9, 2026
69e2188
Forcing hub region processing flow to loop through all available read…
jeet1995 Jan 13, 2026
49deb57
Forcing hub region processing flow to loop through all available read…
jeet1995 Jan 13, 2026
1587ba9
Revert unnecessary changes.
jeet1995 Jan 13, 2026
146f8f0
Fixing compilation errors.
jeet1995 Jan 13, 2026
daaeba2
Fixing compilation errors.
jeet1995 Jan 13, 2026
5ac0021
Stabilizing CI pipeline.
jeet1995 Jan 14, 2026
836983b
Enforce partition-set hub tracking when reads hit 403:3s.
jeet1995 Jan 15, 2026
64cd83a
Add tests for readMany, readAll, query and change feed.
jeet1995 Jan 15, 2026
e74aeca
Add tests for readMany, readAll, query and change feed.
jeet1995 Jan 15, 2026
e6759bb
Add tests for readMany, readAll, query and change feed.
jeet1995 Jan 15, 2026
57121ba
Fixing a query plan caching test.
jeet1995 Jan 16, 2026
a6fb68c
Add hub region processing only request header to CosmosDiagnostics.
jeet1995 Jan 16, 2026
1eec23d
Attempt to make PPAF/PPCB info visible in CosmosDiagnostics.
jeet1995 Jan 17, 2026
337e650
Attempt to make PPAF/PPCB info visible in CosmosDiagnostics.
jeet1995 Jan 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 11 additions & 9 deletions ...-tests/src/test/java/com/azure/cosmos/GlobalPartitionEndpointManagerForPPAFUnitTests.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
package com.azure.cosmos;

import com.azure.cosmos.implementation.AvailabilityStrategyContext;
import com.azure.cosmos.implementation.Configs;
import com.azure.cosmos.implementation.ConnectionPolicy;
import com.azure.cosmos.implementation.CrossRegionAvailabilityContextForRxDocumentServiceRequest;
import com.azure.cosmos.implementation.GlobalEndpointManager;
Expand All @@ -18,7 +17,9 @@
import com.azure.cosmos.implementation.apachecommons.collections.list.UnmodifiableList;
import com.azure.cosmos.implementation.guava25.collect.ImmutableList;
import com.azure.cosmos.implementation.perPartitionAutomaticFailover.GlobalPartitionEndpointManagerForPerPartitionAutomaticFailover;
import com.azure.cosmos.implementation.perPartitionAutomaticFailover.PartitionLevelFailoverInfo;
import com.azure.cosmos.implementation.perPartitionAutomaticFailover.PartitionLevelAutomaticFailoverInfo;
import com.azure.cosmos.implementation.perPartitionAutomaticFailover.PerPartitionAutomaticFailoverInfoHolder;
import com.azure.cosmos.implementation.perPartitionCircuitBreaker.PerPartitionCircuitBreakerInfoHolder;
import com.azure.cosmos.implementation.routing.RegionalRoutingContext;
import com.azure.cosmos.rx.TestSuiteBase;
import org.apache.commons.lang3.tuple.Pair;
Expand All @@ -37,7 +38,6 @@
import java.util.Arrays;
import java.util.Collections;
import java.util.List;
import java.util.Objects;
import java.util.Set;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.ScheduledFuture;
Expand Down Expand Up @@ -148,11 +148,11 @@ public void tryMarkEndpointAsUnavailableForPartitionKeyRange(
String maxExclusive = "BB";
String collectionResourceId = "dbs/db1/colls/coll1";

Field failedRegionalRoutingContextsField = PartitionLevelFailoverInfo.class.getDeclaredField("failedRegionalRoutingContexts");
Field failedRegionalRoutingContextsField = PartitionLevelAutomaticFailoverInfo.class.getDeclaredField("failedRegionalRoutingContexts");

assertThat(failedRegionalRoutingContextsField).isNotNull();

Field currentRegionalContextField = PartitionLevelFailoverInfo.class.getDeclaredField("current");
Field currentRegionalContextField = PartitionLevelAutomaticFailoverInfo.class.getDeclaredField("current");

assertThat(currentRegionalContextField).isNotNull();

Expand Down Expand Up @@ -362,11 +362,11 @@ private static void validateAllRegionsAreNotUnavailableAfterExceptionInLocation(
logger.warn("Handling exception for {}", locationWithFailure.getPath());
globalPartitionEndpointManagerForPerPartitionAutomaticFailover.tryMarkEndpointAsUnavailableForPartitionKeyRange(request, false);

Field failedLocationsField = PartitionLevelFailoverInfo.class.getDeclaredField("failedRegionalRoutingContexts");
Field failedLocationsField = PartitionLevelAutomaticFailoverInfo.class.getDeclaredField("failedRegionalRoutingContexts");

assertThat(failedLocationsField).isNotNull();

Field currentField = PartitionLevelFailoverInfo.class.getDeclaredField("current");
Field currentField = PartitionLevelAutomaticFailoverInfo.class.getDeclaredField("current");

assertThat(currentField).isNotNull();

Expand Down Expand Up @@ -421,8 +421,10 @@ private RxDocumentServiceRequest constructRxDocumentServiceRequestInstance(
false,
collectionLink,
new SerializationDiagnosticsContext()),
new AvailabilityStrategyContext(true, true)
)
new AvailabilityStrategyContext(true, true),
new AtomicBoolean(false),
new PerPartitionCircuitBreakerInfoHolder(),
new PerPartitionAutomaticFailoverInfoHolder())
);

return request;
Expand Down
7 changes: 6 additions & 1 deletion ...-tests/src/test/java/com/azure/cosmos/GlobalPartitionEndpointManagerForPPCBUnitTests.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,12 @@
import com.azure.cosmos.implementation.RxDocumentServiceRequest;
import com.azure.cosmos.implementation.SerializationDiagnosticsContext;
import com.azure.cosmos.implementation.apachecommons.collections.list.UnmodifiableList;
import com.azure.cosmos.implementation.perPartitionAutomaticFailover.PerPartitionAutomaticFailoverInfoHolder;
import com.azure.cosmos.implementation.perPartitionCircuitBreaker.GlobalPartitionEndpointManagerForPerPartitionCircuitBreaker;
import com.azure.cosmos.implementation.perPartitionCircuitBreaker.LocationHealthStatus;
import com.azure.cosmos.implementation.perPartitionCircuitBreaker.LocationSpecificHealthContext;
import com.azure.cosmos.implementation.guava25.collect.ImmutableList;
import com.azure.cosmos.implementation.perPartitionCircuitBreaker.PerPartitionCircuitBreakerInfoHolder;
import com.azure.cosmos.implementation.routing.RegionalRoutingContext;
import org.apache.commons.lang3.tuple.Pair;
import org.mockito.Mockito;
Expand Down Expand Up @@ -1056,7 +1058,10 @@ private RxDocumentServiceRequest constructRxDocumentServiceRequestInstance(
false,
collectionLink,
new SerializationDiagnosticsContext()),
new AvailabilityStrategyContext(false, false)));
new AvailabilityStrategyContext(false, false),
new AtomicBoolean(false),
new PerPartitionCircuitBreakerInfoHolder(),
new PerPartitionAutomaticFailoverInfoHolder()));

return request;
}
Expand Down
Loading
Loading