Support undefined partition keys (PartitionKey.None) across item commands
Maintainer thường phản hồi trong vòng 2 ngày
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 15/100
Hướng nghiên cứu
Start with the shared partition-key conversion that the typed parser uses, and with how mkitem and create item currently infer keys from item content. Before changing anything, reproduce the missing-key case against the emulator and check how print and patch behave. The issue leaves the shell syntax for requesting None open, so agree on that with maintainers first. Done means the acceptance criteria hold, including the warning rules and the tests that use same-id items in the None, Null, and string partitions.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Summary
Cosmos DB supports items whose configured partition-key property is absent. The .NET SDK represents this value with PartitionKey.None, distinct from explicit JSON null, represented by PartitionKey.Null.
Cosmos DB Shell can create and enumerate missing-key items, but no command can explicitly address their logical partition. Consequently, such items cannot be point-read, patched, replaced, transactionally updated, or reliably deleted.
This gap also applies to missing components in hierarchical partition keys. The SDK exposes PartitionKeyBuilder.AddNoneType() for this case.
Current command behavior
| Commands | Current behavior |
|---|---|
mkitem, create item |
Implicitly support create/upsert because the partition key is omitted from CreateItemAsync/UpsertItemAsync and extracted from the document. |
import |
Implicitly supports insert/upsert for the same reason. |
query |
Can discover missing-key items with NOT IS_DEFINED(c.<path>), but cannot use None as an explicit routing value. |
ls, export |
Can enumerate missing-key items through scans/queries. |
watch |
An unscoped feed can observe them; --partition-key cannot select None. |
print |
Cannot point-read an item using PartitionKey.None. PR #248 adds typed scalar and hierarchical keys but no undefined-key representation. |
patch |
Its shared typed parser has no syntax for PartitionKey.None. |
batch |
Cannot open a transactional batch for the None logical partition. |
sproc exec |
Cannot execute a stored procedure against the None logical partition. |
rm |
--partition-key cannot express None; scan-based removal skips items when a configured key path is absent. |
replace |
Rejects an item when any configured partition-key path is absent. |
No shell command currently provides explicit PartitionKey.None support.
SDK and documentation evidence
The SDK documentation defines:
PartitionKey.None: “allows creating and accessing items without a value for partition key.”PartitionKey.Null: creates and accesses items with an explicit null partition-key value.PartitionKeyBuilder.AddNoneType(): adds aNonecomponent to a hierarchical partition key.CreateItemAsyncandUpsertItemAsyncextract the partition key from the item when it is not supplied.IS_DEFINEDdistinguishes a missing property from an explicitly null property.
Microsoft's partitioning guidance recommends a stable, high-cardinality property with a value on every item. Supporting None is therefore primarily needed for legacy, transitional, and exceptional data, not as a recommended data model: partition-key guidance.
Proposed behavior
When a configured partition-key path is absent, treat it as the valid Cosmos DB value PartitionKey.None instead of rejecting or skipping the item.
Define an unambiguous shell representation for explicitly requesting None. It must remain distinct from:
- explicit JSON
null; - the strings
"none"and"undefined"; - an accidentally omitted required argument.
Warning behavior
Warning: Creating or modifying an item with an implicitly inferred
PartitionKey.Nonemust always produce a visible warning.
- Reads using
Noneare valid and must not warn. - Mutations using explicitly requested
Nonemust not warn. - Mutations using implicitly inferred
Nonemust always warn. - Bulk operations must emit one bounded summary warning with the affected item count.
Extend shared partition-key conversion to produce:
PartitionKey.Nonefor a scalar undefined key;PartitionKeyBuilder.AddNoneType()for undefined HPK components.
Commands that derive keys from item content, including mkitem, create item, import, replace, and scan-based rm, should infer None for absent paths. They must warn before or while performing mutations unless the caller explicitly acknowledged None.
Commands with explicit partition targeting, including print, patch, batch, sproc, rm, and partition-scoped watch, should accept the new explicit representation. Because these requests are intentional, they should not warn.
Acceptance criteria
- Missing scalar and hierarchical key components are converted to the SDK's
Nonerepresentation instead of being rejected or skipped. - Reads using inferred or explicit
Nonedo not warn. - Every mutation using implicitly inferred
Noneemits a warning. - Mutations using explicitly requested
Nonedo not warn. - Bulk mutations produce one bounded summary warning with an accurate item count.
- Create fixtures with the same
idin theNone,Null, and string logical partitions and prove each command addresses only the intended item. - Cover scalar and hierarchical keys with one or more missing components.
- Point-read, patch, replace, delete, batch, stored-procedure, and scoped-watch operations use the SDK's
Nonerepresentation. - Preserve literal strings resembling the chosen
Nonesentinel. - Keep omitted required arguments distinct from an explicit
Nonevalue. - Add shell parser, command, MCP schema/call, warning, and emulator integration coverage.
- Document the syntax, warning behavior, distinction from
null, and Microsoft's recommendation to use a present, stable, high-cardinality partition-key value. - Coordinate the
printimplementation with #248.
- Ngôn ngữ chính
- C#
- Star
- 4
- Fork
- 7
- Merge trung bình
- 2 ngày 40 phút
- Pull request đã merge (30 ngày)
- 30
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của Azure/CosmosDBShell
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
Azure/CosmosDBShell#240 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
automation P1
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 45/100
Azure/CosmosDBShell#178 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Lightweight headless CI distribution (no MCP, LSP, interactive UI)Có thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mởautomation P1
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
Azure/CosmosDBShell#175 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
agentic enhancement P0
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 55/100
Azure/CosmosDBShell#153 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
Azure/CosmosDBShell#104 · 8 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của Azure/CosmosDBShell
Issue tương tự
-
needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
7.0
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
unoplatform/uno.extensions#3253 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Sector-Vestige/Sector-Vestige#476 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
PowerShell/platyPS#868 ·
-
S: Untriaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Trauma-Station/Trauma-Station#5058 ·
Maintainer thường phản hồi trong vòng 1 ngày