-
Notifications
You must be signed in to change notification settings - Fork 977
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
Platform connections: Add new data platform adapter for CrateDB #6564
Conversation
Hello!👋 Thanks for contributing to the dbt product documentation and opening this pull request! ✨ |
@amotl is attempting to deploy a commit to the dbt-labs Team on Vercel. A member of the Team first needs to authorize it. |
1c8b4ac
to
9b288ca
Compare
dbt-cratedb2
dbt-cratedb2
4f64666
to
a77cf43
Compare
Dear @mirnawong1 and @matthewshaver, I just wanted to check if you need anything from my end to complete this patch, for example by improving it? With kind regards, |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
hey @amotl, thanks so much for opening this up and happy to look at this for you! |
456830f
to
1187cd9
Compare
4158188
to
867d96d
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.
Thank you very much for the excellent guidance. 🌻 |
great stuff, thank you @amotl and @amychen1776 ! happy to merge this ? |
hey @amotl , have enabled auto merge. any chance you can update this pr with the base branch? |
Head branch was pushed to by a user without write access
@mirnawong1: Thanks. I think the job still needs attention once more. Because I am an outsider, auto-merge was disabled again.
|
- Fix a typo s/specifc/specific/ - Add third variant why no documentation about platform-specific configuration settings is provided, by referring to vendor documentation pages
The Python package is called `dbt-cratedb2`. Package: https://pypi.org/project/dbt-cratedb2/ Repository: https://github.com/crate/dbt-cratedb2 Documentation: https://cratedb.com/docs/guide/integrate/dbt/
Head branch was pushed to by a user without write access
Wonderful. We made it! Thank you again. |
About
First things first, thanks a stack for conceiving and maintaining dbt. 💯
This patch adds information about the database adapter for the CrateDB database to dbt's documentation. Because CrateDB is compatible with PostgreSQL, it builds heavily upon the dbt-postgres adapter.
It was tremendously helpful for us to get started by gradually working through its test suite to progressively make test cases succeed on CrateDB, so we would like to explicitly send kudos to @mikealfare and all contributors.
Preview
CrateDB setup
Details
Package: https://pypi.org/project/dbt-cratedb2/
Repository: https://github.com/crate/dbt-cratedb2
Documentation: https://cratedb.com/docs/guide/integrate/dbt/
Checklist
ADDING OR REMOVING PAGES:
website/sidebars.js
website/vercel.json
npm run build
to update the links that point to deleted pages