Azure / azure-cosmosdb-bulkexecutor-dotnet-getting-started

Bulk Executor Utility for Azure Cosmos DB .NET SQL API
MIT License
66 stars 41 forks source link

Improvement request for BulkExecutor constructor #42

Open Ozzian opened 5 years ago

Ozzian commented 5 years ago

I couldn't find the implementation of the BulkExecutor, so i'm posting this here. (Can someone refer me to the correct github repository?)

Hear me oh gods of BulkExecutor, angels of CosmosDb! Hear me pray!

The constructor of BulkExecutor requires DocumentClient and DocumentCollection. That makes testing quite tricky. Can we have the constructor defined with interfaces ( IDocumentClient )? Also I would like to have the DocumentCollection defined elsewhere from the constructor, like when initializing the bulk operations or so...