-
Notifications
You must be signed in to change notification settings - Fork 33
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
feat: implement datafusion TableProviderFactory
#162
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #162 +/- ##
==========================================
- Coverage 90.53% 90.44% -0.09%
==========================================
Files 19 19
Lines 961 973 +12
==========================================
+ Hits 870 880 +10
- Misses 91 93 +2 ☔ View full report in Codecov by Sentry. |
@xushiyan can I get another CI run, I added a test for when options are passed to |
@kazdy i'll push quick fixes and then freeze the code for release |
d5254df
to
caf1233
Compare
caf1233
to
cbd1c99
Compare
TableProviderFactory
Description
resolves: #150
This allows datafusion users to register an external hudi table and query it like so:
How are the changes test-covered