-
Notifications
You must be signed in to change notification settings - Fork 48
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
[sql-cli] Implement load file operator #1258
Conversation
00b59d1
to
0e27351
Compare
Codecov ReportBase: 94.13% // Head: 95.08% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1258 +/- ##
==========================================
+ Coverage 94.13% 95.08% +0.94%
==========================================
Files 17 71 +54
Lines 597 3296 +2699
Branches 67 380 +313
==========================================
+ Hits 562 3134 +2572
- Misses 22 100 +78
- Partials 13 62 +49
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. |
2213b22
to
4214530
Compare
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 👍
Let's only fix the remaining failed SQL CLI test.
All other failing tests are related to python-sdk. :(
- move pytest ini options to pyproject toml - refactor code
- refactor code
60748d3
to
ce52863
Compare
Description
What is the current behavior?
Currently, we are not having a way for users to load data into databases.
closes: #1227
What is the new behavior?
Add
load_file
/LoadFileOperator
to the sql-cli.Does this introduce a breaking change?
No.
Checklist