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

Add CosmosDB parent-child relationship bits. #2912

Open
wants to merge 8 commits into
base: release-9.2
Choose a base branch
from

Conversation

IEvangelist
Copy link
Member

@IEvangelist IEvangelist commented Apr 1, 2025

@IEvangelist IEvangelist requested a review from captainsafia April 1, 2025 19:31
@IEvangelist IEvangelist marked this pull request as draft April 1, 2025 19:32
@IEvangelist IEvangelist changed the title Add CosmosDB parent-child relationship bits. Part of #2907 Add CosmosDB parent-child relationship bits. Apr 1, 2025
@IEvangelist IEvangelist marked this pull request as ready for review April 2, 2025 19:42
app.Run();
```

If you have multiple containers under the same database connection, you can use the `AddAzureCosmosDatabase` API to attach multiple containers under the same database connection. All child containers share the same <xref:Azure.Cosmos.CosmosClient> and database connection. This strategy is useful when associating the same <xref:Azure.Cosmos.CosmosClientOptions> with multiple containers. Consider the following alternative code, to register multiple containers under the same database connection:
Copy link
Member

Choose a reason for hiding this comment

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

Should we move this section earlier to where we talk about AddCosmosDatabase. Right now, we allude to the multiple container thing in the database section but don't get back to talking about it under here.

Copy link
Member Author

@IEvangelist IEvangelist Apr 3, 2025

Choose a reason for hiding this comment

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

Those are separate sections, hosting vs client integrations, no? I could link to one from the other, but if they're together, then we're merging the two concepts/integrations together.

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.

2 participants