-
Notifications
You must be signed in to change notification settings - Fork 934
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
Nfiann-prehook-emptyflag #6228
base: current
Are you sure you want to change the base?
Nfiann-prehook-emptyflag #6228
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
hey @nataliefiann , good job! have some suggestions on my end to keep it more concise and address the actual concept
…s/docs.getdbt.com into nfiann-prehook-emptyflag
@@ -0,0 +1,17 @@ | |||
#### The render method | |||
|
|||
The `.render()` method is generally used to resolve or evaluate Jinja expressions (such as `{{ source(...) }}`) during runtime. When a pre-hook or post-hook contains a dynamic reference, such as a table or column, dbt might not automatically resolve the reference correctly, particularly when certain flags (such as `--empty`) are applied. |
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.
sugg removing the first paragraph to simplify the explanation since the 2nd paragraph already covers the key point about using .render() with the --empty flag
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.
small sugg and approved
I've created this PR to properly capture render process for SQL compilation error and used reusable in build and pre-hook doc
What are you changing in this pull request and why?
Checklist