-
Notifications
You must be signed in to change notification settings - Fork 2.2k
[Gateway V1/V2]: Enable ReadConsistencyStrategy for Gateway V1 and Gateway V2 #48787
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.
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
Open
jeet1995
wants to merge
47
commits into
Azure:main
Choose a base branch
from
jeet1995:squad/48094-readconsistency-header-propagation
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
ddabe92
Add ReadConsistencyStrategy RNTBD header and enable for all connectio…
jeet1995 faa9ccf
Fix: prevent x-ms-consistency-level rewrite in gateway mode when RCS …
jeet1995 6c0913c
Fix: strip x-ms-consistency-level when ReadConsistencyStrategy is set…
jeet1995 a6c0b50
Fix: change ReadConsistencyStrategy RNTBD token type from String to B…
jeet1995 db3ee0c
Update E2E tests: dynamic database/container creation, use TestConfig…
jeet1995 3f80dac
Fix E2E tests for serverless accounts: remove throughput parameter (#…
jeet1995 3fe5064
Comprehensive E2E tests for ReadConsistencyStrategy across all reques…
jeet1995 adc17bd
Fix test-resources.json: bump Cosmos DB API version to 2023-04-15
jeet1995 8e262cb
Test: revert to 2022-08-15 with EnableNoSQLVectorSearch removed
jeet1995 6e84b99
Fix test-resources.json: bump API version to 2023-04-15 for EnableNoS…
jeet1995 7991337
Test: re-verify 2022-08-15 + EnableNoSQLVectorSearch (expect 400)
jeet1995 e00f5bb
Centralize consistency flag contention resolution in RxGatewayStoreModel
jeet1995 6aa54a5
Add spy-wire tests to verify ReadConsistencyStrategy headers on the w…
jeet1995 ce56e3e
Fix RNTBD header tests for Byte type + add ThinClient RNTBD spy-wire …
jeet1995 df06da1
Extract RCS HTTP spy-wire tests into standalone serverless-safe class…
jeet1995 7e542aa
Rename RCS abbreviation to readConsistencyStrategy throughout (#48094)
jeet1995 d4357e3
Add spy-wire, mock-pipeline, and thin-client GLOBAL_STRONG tests (#48…
jeet1995 4458158
Fix unused strategy parameter in RntbdReadConsistencyStrategyHeaderTe…
jeet1995 e3f4a84
Remove test-output build artifacts from tracking (#48094)
jeet1995 486347e
Replace FQN GatewayConnectionConfig with import (#48094)
jeet1995 879552b
Replace FQN HttpRequest with import in RntbdReadConsistencyStrategyHe…
jeet1995 f05b3c3
Eliminate simulated resolveEffectiveConsistencyHeaders in tests (#48094)
jeet1995 c2df4d4
Add operation policy E2E tests and document RntbdReadConsistencyStrat…
jeet1995 8a5b477
Add thin client operation policy E2E tests for readConsistencyStrateg…
jeet1995 2ecdbd8
Fix RxGatewayStoreModelTest mock verification for retry behavior (#48…
jeet1995 9bbbcb9
Use Gateway V1/V2 terminology in customer-facing docs (#48094)
jeet1995 f078479
Fix CHANGELOG/Javadoc: Direct mode was already supported before this …
jeet1995 83fc115
Add readAllItems E2E tests for readConsistencyStrategy on Gateway V1 …
jeet1995 ff18029
Replace HashMap copy in applySessionToken with pure isEffectiveSessio…
jeet1995 8454f70
Refactoring
jeet1995 d6ba570
Consolidate ReadConsistencyStrategy test coverage into unified spy wi…
jeet1995 e5418f1
Refactoring
jeet1995 a275926
Fix unified E2E test: standalone pattern without TestSuiteBase inheri…
jeet1995 3f21d1d
Move ReadConsistencyStrategy test files to com.azure.cosmos.rx package
jeet1995 3db0af8
Remove v2Available/gatewayV2Available gates and probe logic
jeet1995 f9865c5
Rename E2E and spy wire tests to call out gateway scope
jeet1995 b0ed241
Replace brute-force RNTBD byte scanners with RntbdRequest.decode in u…
jeet1995 a81fbf3
Refactoring
jeet1995 4584597
Improve ReadConsistencyStrategy javadoc for customer readability
jeet1995 b0f6d2b
Refactoring
jeet1995 52da3ae
Merge remote-tracking branch 'upstream/main' into squad/48094-readcon…
jeet1995 0b7f706
Add Direct mode to unified E2E test for ReadConsistencyStrategy
jeet1995 3bbd63f
Address review: treat DEFAULT as transparent, add SESSION E2E tests
jeet1995 8e5013e
Refactoring
jeet1995 f0dfc5c
Refactoring
jeet1995 6fcefa0
Fixing tests.
jeet1995 af4b7c5
Revert ReadConsistencyStrategy.java javadoc to match Azure/main
jeet1995 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
431 changes: 431 additions & 0 deletions
431
...smos/implementation/directconnectivity/rntbd/RntbdReadConsistencyStrategyHeaderTests.java
Large diffs are not rendered by default.
Oops, something went wrong.
719 changes: 719 additions & 0 deletions
719
...cosmos-tests/src/test/java/com/azure/cosmos/rx/GatewayReadConsistencyStrategyE2ETest.java
Large diffs are not rendered by default.
Oops, something went wrong.
486 changes: 486 additions & 0 deletions
486
...os-tests/src/test/java/com/azure/cosmos/rx/GatewayReadConsistencyStrategySpyWireTest.java
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.