-
Notifications
You must be signed in to change notification settings - Fork 234
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 update-table command to Delta Sharing CLI #568
Conversation
Codecov ReportBase: 75.49% // Head: 75.72% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #568 +/- ##
==========================================
+ Coverage 75.49% 75.72% +0.23%
==========================================
Files 55 55
Lines 4823 4906 +83
==========================================
+ Hits 3641 3715 +74
- Misses 1182 1191 +9
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
Also, all databricks-cli commits need to be signed. https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits
063548f
to
7bde91e
Compare
…ions, use partition spec schema as json
…able name or shared-as name, refactor code so common shared data object are in a decorator function, add more tests
…e formatting issues
… make the help statements more generic
How about start version for CDF? |
Added it! Asked Lin and he was good with adding it! |
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, bar a tiny comment.
Ready to go otherwise.
This PR adds the following command to Delta Sharing CLI: