-
Notifications
You must be signed in to change notification settings - Fork 55
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
request for enhanced properties feature #19
Comments
@bachng2017 thank you for the feedback. Regarding the 1st point, please do describe in detail your use case with concrete code snippets. Same thing for the 2st point. |
From dbt slack you may make use of dbt macros for adding dynamic functionality to the
or simply do this:
|
Let me clarify
But with the current implementation, we need to add configuration for all
But let image we have more models, more Notes that in this case
for .2 we are now have a small patch for adapters.sql as following. It works in our case but not sure for other cases
|
The properties feature the implement recently is very useful but it would be better if it could support following use-cases:
The text was updated successfully, but these errors were encountered: