Skip to content

Hierarchical partition keys guidance on Partial PK#363

Merged
v-dirichards merged 1 commit intoMicrosoftDocs:mainfrom
ananth7592:ananth/fix-hpk-query-docs
Apr 21, 2026
Merged

Hierarchical partition keys guidance on Partial PK#363
v-dirichards merged 1 commit intoMicrosoftDocs:mainfrom
ananth7592:ananth/fix-hpk-query-docs

Conversation

@ananth7592
Copy link
Copy Markdown
Contributor

@ananth7592 ananth7592 commented Apr 14, 2026

No description provided.

@ananth7592 ananth7592 changed the title Hierarhcical partition keys guidance on Partial PK Hierarchical partition keys guidance on Partial PK Apr 14, 2026
@prmerger-automator prmerger-automator Bot requested a review from deborahc April 14, 2026 19:48
@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented Apr 14, 2026

@ananth7592 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@prmerger-automator prmerger-automator Bot requested a review from markjbrown April 14, 2026 19:48
@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented Apr 14, 2026

@ananth7592 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 14, 2026

Learn Build status updates of commit ea7d712:

✅ Validation status: passed

File Status Preview URL Details
articles/cosmos-db/hierarchical-partition-keys.md ✅Succeeded

For more details, please refer to the build report.

Praveen-Msft
Praveen-Msft reviewed Apr 14, 2026
View reviewed changes
Comment thread articles/cosmos-db/hierarchical-partition-keys.md
@v-regandowner
Copy link
Copy Markdown
Contributor

v-regandowner commented Apr 14, 2026

@deborahc

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged C+L Pull Request Review Team label label Apr 14, 2026
@v-regandowner v-regandowner requested a review from Copilot April 14, 2026 20:14
Copilot AI reviewed Apr 14, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds guidance to the hierarchical partition keys documentation to clarify query routing behavior when only a prefix of the hierarchical partition key is provided.

Changes:

  • Documented that partial-prefix query routing requires the prefix to be present in the WHERE clause
  • Warned that relying only on PartitionKeyBuilder may not ensure efficient routing and can increase RU costs

Comment thread articles/cosmos-db/hierarchical-partition-keys.md Outdated
@ananth7592 ananth7592 force-pushed the ananth/fix-hpk-query-docs branch from ea7d712 to f865b0e Compare April 14, 2026 22:59
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 14, 2026

Learn Build status updates of commit f865b0e:

✅ Validation status: passed

File Status Preview URL Details
articles/cosmos-db/hierarchical-partition-keys.md ✅Succeeded

For more details, please refer to the build report.

@markjbrown
Copy link
Copy Markdown
Contributor

markjbrown commented Apr 21, 2026

#sign-off

@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented Apr 21, 2026

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @deborahc

@markjbrown
Copy link
Copy Markdown
Contributor

markjbrown commented Apr 21, 2026

@v-regandowner, the original author is no longer at MS. Can you please fork this PR and change the author of this article to me then merge this change?

@v-dirichards
Copy link
Copy Markdown
Contributor

v-dirichards commented Apr 21, 2026 •
edited
Loading

@v-regandowner, the original author is no longer at MS. Can you please fork this PR and change the author of this article to me then merge this change?

@markjbrown We can accept this note as a proper sign-off.

@v-dirichards v-dirichards merged commit f63ec38 into MicrosoftDocs:main Apr 21, 2026
2 checks passed
@v-dirichards
Copy link
Copy Markdown
Contributor

v-dirichards commented Apr 21, 2026

@markjbrown In the public repos, we typically don't update the ms.author. Could you open a PR in the private repo to change the article's owner?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@deborahc deborahc Awaiting requested review from deborahc

+4 more reviewers

@markjbrown markjbrown markjbrown approved these changes

@kirankumarkolli kirankumarkolli kirankumarkolli approved these changes

@yash2710 yash2710 yash2710 approved these changes

@Praveen-Msft Praveen-Msft Praveen-Msft approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@deborahc deborahc

Labels

aq-pr-triaged C+L Pull Request Review Team label azure-cosmos-db/svc Change sent to author do-not-merge nosql/subsvc

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@ananth7592 @v-regandowner @markjbrown @v-dirichards @kirankumarkolli @yash2710 @Praveen-Msft @deborahc