-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
bef8c4c
to
4dbb739
Compare
4dbb739
to
41d664d
Compare
8cd8fea
to
3946c15
Compare
@JeppeBlixen Do you want to review before we merge? We are missing some good documentation for the class. I hope you can get started with the example in the unittest. |
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.
It looks good to me, some nice features around the partitioning. This could also be used for other time partitioned file sources, for example json. Would it make sense to split this? One part with generalized logic about the partitioning, and another more related to the eventhub with the validation and table creation.
@JeppeBlixen That's great input for the next PR. For now I want to complete this and get on with my vacation. We can talk again after easter. |
No description provided.