We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
To allow all schemas to be PR-specific, replace any hardcoded schema names with the dynamic target.schema that comes from the dbt connection profile.
target.schema
In our CI tests, we have steps like this:
dbt --warn-error run-operation create_source_table
Within that macro, we create a schema named codegen_integration_tests__data_source_schema (which is a hard-coded name):
codegen_integration_tests__data_source_schema
dbt-codegen/integration_tests/macros/operations/create_source_table.sql
Lines 3 to 6 in fdc998c
This could lead to permissions errors if different database users are trying to modify this schema.
This will benefit maintainers that want CI tests to be reliable and robust rather than finicky.
In particular, I think it will help overcome permissions errors faced in these PRs:
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Describe the feature
To allow all schemas to be PR-specific, replace any hardcoded schema names with the dynamic
target.schema
that comes from the dbt connection profile.Additional context
In our CI tests, we have steps like this:
Within that macro, we create a schema named
codegen_integration_tests__data_source_schema
(which is a hard-coded name):dbt-codegen/integration_tests/macros/operations/create_source_table.sql
Lines 3 to 6 in fdc998c
This could lead to permissions errors if different database users are trying to modify this schema.
Who will this benefit?
This will benefit maintainers that want CI tests to be reliable and robust rather than finicky.
In particular, I think it will help overcome permissions errors faced in these PRs:
Are you interested in contributing this feature?
The text was updated successfully, but these errors were encountered: