-
Notifications
You must be signed in to change notification settings - Fork 68
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
Cosmos V4: Reduce initialization confusion #208
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bartelink from what I can tell, this removes the ability for the consumer to provide their own implementation of ContainerGateway
, which was the whole point of the client abstractions work. Unless I'm not seeing something?
Apologies; Its possible that it's not wired through, and I agree its a pretty big gap |
The above exposes a |
More stuff will likely happen to more closely align with https://azure.github.io/azure-sdk/dotnet_introduction.html before a V3 beta... |
Merging as agreed out of band with @ylibrach - some naming and signature changes will follow as independent PRs |
re #197 This addresses a longstanding mess in Cosmos.fs wrt how initialization is managed
@ylibrach this has placeholder names that would benefit from some pairing if you're up for it ;)