Support undefined partition keys (PartitionKey.None) across item commands
Les mainteneurs répondent en général sous 2 jours
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 15/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- C#
- Étoiles
- 4
- Forks
- 7
- Merge moyen
- 2 j 40 min
- PR mergées (30 j)
- 30
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Azure/CosmosDBShell
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
Azure/CosmosDBShell#240 ·
Les mainteneurs répondent en général sous 2 jours
-
automation P1
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 45/100
Azure/CosmosDBShell#178 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
Lightweight headless CI distribution (no MCP, LSP, interactive UI)Peut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverteautomation P1
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
Azure/CosmosDBShell#175 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
agentic enhancement P0
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
Azure/CosmosDBShell#153 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
Azure/CosmosDBShell#104 · 8 commentaires ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de Azure/CosmosDBShell
Issues similaires
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
ClickHouse/clickhouse-cs#689 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 7 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 65/100
vchelaru/FlatRedBall2#1481 ·
Les mainteneurs répondent en général sous 1 jour
-
ai-chat bug trivial
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
area-signalr
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
dotnet/aspnetcore#69842 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour