Skip to content
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

Merged
merged 63 commits into from
Feb 13, 2021

Conversation

SrinikhilReddy
Copy link
Contributor

@SrinikhilReddy SrinikhilReddy commented Oct 22, 2020

Enable subpartitioning feature to be available as public preview.

Description

Replacing the existing internal preprocessor directives with preview for subpartitioning feature code.

## Type of change

- [x] New feature (non-breaking change which adds functionality)

SrinikhilReddy and others added 30 commits June 23, 2020 00:12
@SrinikhilReddy SrinikhilReddy changed the title Enable Subpartitioning feature to be available as public preview. [Internal] Refactor code to enable Subpartitioning feature to be available as public preview. Oct 27, 2020
@SrinikhilReddy SrinikhilReddy changed the title [Internal] Refactor code to enable Subpartitioning feature to be available as public preview. [Internal] Refactor code to enable subpartitioning feature available as public preview. Oct 27, 2020
@SrinikhilReddy SrinikhilReddy changed the title [Internal] Refactor code to enable subpartitioning feature available as public preview. [Internal] Refactor code to replace internal flags with preview flags for subpartitioning feature Oct 27, 2020
@SrinikhilReddy SrinikhilReddy changed the title [Internal] Refactor code to replace internal flags with preview flags for subpartitioning feature [Internal] Subpartitioning: Refactor code to replace internal flags with preview flags for public preview Oct 27, 2020
@SrinikhilReddy SrinikhilReddy changed the title [Internal] Subpartitioning: Refactor code to replace internal flags with preview flags for public preview [Internal] Subpartitioning: Refactors code to replace internal flags with preview flags for public preview Oct 27, 2020
@SrinikhilReddy SrinikhilReddy changed the title [Internal] Subpartitioning: Refactors code to replace internal flags with preview flags for public preview Subpartitioning: Refactors code to replace internal flags with preview flags for public preview Oct 28, 2020
@j82w
Copy link
Contributor

j82w commented Jan 4, 2021

Closing since there has been no update for over 30 days. Please re-open if necessary.

@j82w j82w closed this Jan 4, 2021
@SrinikhilReddy
Copy link
Contributor Author

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.

@SrinikhilReddy SrinikhilReddy reopened this Jan 7, 2021
@@ -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
Copy link
Member

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

Copy link
Member

@kirankumarkolli kirankumarkolli left a 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

@ghost
Copy link

ghost commented Dec 15, 2021

Closing due to in-activity, pease feel free to re-open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants