-
Notifications
You must be signed in to change notification settings - Fork 982
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 documentation about the dbt-risingwave adapter #4747
Conversation
Hello!👋 Thanks for contributing to the dbt product documentation and opening this pull request! ✨ |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Somehow the preview failed. I couldn't log in to Vercel to see what happened. Please help take a look. Appreciate your help. :) |
@runleonarun Could you please take a look at this PR? I wasn't able to see why the deployment failed in this case. Thanks a lot! |
Yes! @hengm3467 Looking now! |
website/docs/docs/core/connect-data-platform/risingwave-setup.md
Outdated
Show resolved
Hide resolved
website/docs/docs/core/connect-data-platform/risingwave-setup.md
Outdated
Show resolved
Hide resolved
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 @hengm3467, the build failed with a formatting error. I left a comment where the error is coming from. Adjusting that should get the build to complete successfully
website/docs/docs/core/connect-data-platform/risingwave-setup.md
Outdated
Show resolved
Hide resolved
@hengm3467 if you could make Amy's suggested changes, we can get this merged for you. |
Co-authored-by: Amy Chen <[email protected]>
@amychen1776 @runleonarun Thanks a lot for your help! I've made the changes suggested by Amy. Please take a look again. |
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.
@JKarlavige I see the formatting issue was fixed. Could you please confirm and, if that's the case, approve this PR? Thanks a lot! :) |
Yep the formatting was fixed! @hengm3467 One tiny thing. You might want to add the adapter to this page too. Then I can approve and merge! |
Head branch was pushed to by a user without write access
@runleonarun I have just added the adapter to the Community adapters page. Please take a look and let me know if any questions. Thanks for your help! :) |
Looks awesome! Thanks for your work @hengm3467 and @amychen1776! |
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.
LGTM!
What are you changing in this pull request and why?
Add documentation about the dbt-risingwave adapter. RisingWave is a streaming database that allows users to process and manage streaming data with a PostgreSQL-style experience. For more information about the company and product, see https://risingwave.com/.
The purpose of this adapter is to enable RisingWave users to leverage the powerful capabilities of dbt to manage their stream processing jobs and pipelines in RisingWave. We consider dbt as the right way to unlock stream processing with our product.
This adapter is available on GitHub repo and PyPl.
Checklist
Adding or removing pages (delete if not applicable):
website/sidebars.js
/docs/core/connect-data-platform/risingwave-setup.md