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(duckdb-driver): Add support for local fs duckdb database path #7799
Conversation
Add support for duckdb database file path
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 8 Ignored Deployments
|
@JichaoS Thanks for the contribution! GitHub tells this PR conflicts with the main branch. Could you please resolve that? |
Done! |
Parameter must be undefined if no fields.
Fix linter errors
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7799 +/- ##
===========================================
- Coverage 67.88% 47.95% -19.94%
===========================================
Files 345 154 -191
Lines 58124 20969 -37155
Branches 5394 5394
===========================================
- Hits 39459 10056 -29403
+ Misses 17918 10166 -7752
Partials 747 747
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Check List
Issue Reference this PR resolves
#7653
Description of Changes Made (if issue reference is not provided)
Creates a new env var that allows users to define a duckdb local file.
Testing coverage is on par with existing CUBEJS_DB_DUCKDB_MOTHERDUCK_TOKEN