-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat(dremio-driver): Add Dremio Cloud Support #8956
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 8 Skipped Deployments
|
KSDaemon
left a comment
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.
Hi @maladroitthief! Thanks for your contribution!
I left some comments how to make it cleaner and more aligned with the rest codebase.
|
And if you don't mind — please sync with the latest master (it would be easy, and no conflicts, but there are some updates related to ci) |
- Adds support to cubejs-dremio-driver for Dremio Cloud - Adds support for personal access tokens for Dremio Server
- Adds the dremioAuthToken config to support the dremio cloud driver
068560b to
15e745e
Compare
KSDaemon
left a comment
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.
👍🏻
Check List
Issue Reference this PR resolves
closes #8166
Description of Changes Made (if issue reference is not provided)