-
Notifications
You must be signed in to change notification settings - Fork 188
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 Databricks SQL Connector #133
Comments
Hey @DanFitzgibbon! Thanks a lot for the issue - we definitely want to have a DataBricks Warehouse connector and would love your contribution. If you're interested in trying it out, please have a look at #127 and #107 - they implement connectors for Snowflake and MSSQL so that could be a good inspiration. Implementing a connector is all about following the existing patterns - it isn't something that's really that hard (in terms of complexity) tbh Lmk what you think - happy to assign this to you if you want |
Heya Lucas! Yup, feel free to assign me, please - will take a look at both PRs and see where that takes me. 😊 Thanks a bunch! |
Heya, PR ready for review! conscious there's another PR adding MonetDB - if you want to merge that given it's an older issue, I'll just rebase my branch with main once it's merged. |
(Am interested in developing this on my own but any help is of course very welcome!)
For clarity's sake: the idea is to have a Databricks Warehouse connector.
The text was updated successfully, but these errors were encountered: