-
Notifications
You must be signed in to change notification settings - Fork 498
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
Subpartitioning: Refactors code to replace internal flags with preview flags for public preview #1952
Conversation
…ilReddy/azure-cosmos-dotnet-v3 into users/nanarava/multihash
…ilReddy/azure-cosmos-dotnet-v3 into users/nanarava/multihash
Closing since there has been no update for over 30 days. Please re-open if necessary. |
This PR was closed because of a lack of Java support for subpartitioning. Java PR is now out, Azure/azure-sdk-for-java#18503. Reopening this PR for merge consideration. |
@@ -11,7 +11,7 @@ namespace Microsoft.Azure.Cosmos | |||
/// <summary> | |||
/// Represents a partition key value list in the Azure Cosmos DB service. | |||
/// </summary> | |||
#if INTERNAL || SUBPARTITIONING | |||
#if PREVIEW |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please cover in the code docs that its an ordered pk collection
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets please ensure that JAVA and .NET API's are in sync.
JAVA has new comments and if deviates then similar changes are needed in .NET
Updating files for PR merge readiness
Closing due to in-activity, pease feel free to re-open. |
Enable subpartitioning feature to be available as public preview.
Description
Replacing the existing internal preprocessor directives with preview for subpartitioning feature code.