-
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
Added support for CosmosDB v4 extension #175
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #175 +/- ##
==========================================
+ Coverage 90.28% 90.51% +0.23%
==========================================
Files 53 53
Lines 3046 3122 +76
Branches 809 815 +6
==========================================
+ Hits 2750 2826 +76
Misses 219 219
Partials 77 77
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
LGTM
Thanks for the fast turn over!
@@ -106,3 +107,103 @@ def __init__(self, | |||
self.database_name = database_name | |||
self.collection_name = collection_name | |||
super().__init__(name=name, data_type=data_type) | |||
|
|||
|
|||
class CosmosDBInput(InputBinding): |
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.
In the comments, could you please add a description of what is the difference between CosmosDBTriggerV3
vs CosmosDBTrigger
so that that info is in the code.
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.
Added
CosmosDB data is changed. | ||
indexing model. This decorator will work only with extension bundle 2.x | ||
or 3.x. For additional details, please refer | ||
https://github.com/Azure/azure-functions-python-worker/issues/1222. |
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.
Why not put an aka.ms link or mslearn link? aka.ms can be tracked.
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.
Added
All optional fields will be given default value by function host when | ||
they are parsed by function host. | ||
|
||
Ref: https://aka.ms/azure-function-binding-cosmosdb-v2 |
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.
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.
Similar comments to the other decorators as well.
Added support for cosmos db v4 extension for the new programming model.
New decorators for cosmosdb:
Extension 2.x and 3.x
Extension 4.x and above
The new v4 cosmos db extension installation can be found here
The latest support configurations for the new programming model can be found here