Revert "Add GlobalSecondaryIndexDefinition for GSI container support (#47468)"#48054
Closed
simorenoh wants to merge 1 commit into
Closed
Revert "Add GlobalSecondaryIndexDefinition for GSI container support (#47468)"#48054simorenoh wants to merge 1 commit into
simorenoh wants to merge 1 commit into
Conversation
|
Azure Pipelines: Successfully started running 2 pipeline(s). 8 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
1 similar comment
|
Azure Pipelines: Successfully started running 2 pipeline(s). 8 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
Member
Author
|
/azp run python - cosmos - tests |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Contributor
There was a problem hiding this comment.
Pull request overview
Reverts the unreleased Cosmos DB GSI container support, restoring the package to its pre-#47468 state.
Changes:
- Removes the GSI API and sync/async integration.
- Deletes GSI tests and pipeline configuration.
- Restores changelog and API snapshots.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
sdk/cosmos/tests.yml |
Removes GSI credentials. |
sdk/cosmos/live-platform-matrix.json |
Removes the GSI test matrix. |
sdk/cosmos/azure-cosmos/tests/test_global_secondary_index.py |
Deletes GSI unit tests. |
sdk/cosmos/azure-cosmos/tests/test_global_secondary_index_live.py |
Deletes GSI live tests. |
sdk/cosmos/azure-cosmos/pytest.ini |
Removes the GSI marker. |
sdk/cosmos/azure-cosmos/CHANGELOG.md |
Removes the unreleased feature entry. |
sdk/cosmos/azure-cosmos/azure/cosmos/database.py |
Removes synchronous GSI parameters and serialization. |
sdk/cosmos/azure-cosmos/azure/cosmos/aio/_database.py |
Removes asynchronous GSI support. |
sdk/cosmos/azure-cosmos/azure/cosmos/_global_secondary_index.py |
Deletes the GSI definition class. |
sdk/cosmos/azure-cosmos/azure/cosmos/__init__.py |
Removes the public GSI export. |
sdk/cosmos/azure-cosmos/api.metadata.yml |
Updates the API snapshot hash. |
sdk/cosmos/azure-cosmos/api.md |
Removes GSI API entries. |
mbhaskar
approved these changes
Jul 14, 2026
Member
Author
|
don't merge this yet - trying to reconcile changes here first as the ideal scenario: #48038 |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
This reverts PR #47468 ("Add GlobalSecondaryIndexDefinition for GSI container support").
The original PR was squash-merged as commit
8073e65288a68edf18b22c4a3932b89b9a5f383e. This PR reverts that single commit, restoring theazure-cosmospackage to its prior state.What this reverts
GlobalSecondaryIndexDefinitionclass (azure/cosmos/_global_secondary_index.py)global_secondary_index_definitionkeyword fromcreate_container,create_container_if_not_exists, andreplace_container(sync and async)pytest.ini,tests.yml,live-platform-matrix.json)CHANGELOG.md,api.md, andapi.metadata.ymlNet: 2 insertions, 645 deletions across 12 files — a clean inverse of the original change. No dangling references to the removed symbols remain.