-
Notifications
You must be signed in to change notification settings - Fork 179
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
quoting config not working with 1.8 #1075
Conversation
We do this in two spots, so this line would also need updating. |
…on, update macros to have condtion to check based on type
schema_relation = {"database": database, "schema": schema} | ||
kwargs = {"schema_relation": schema_relation} | ||
schema_relation = SnowflakeRelation.create(database=database, schema=schema) | ||
kwargs = {"schema_relation": schema_relation.render()} |
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.
Is there a reason to render here? Are we testing if the macro can take a relation object or a string (or both I suppose)?
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.
you can't pass a proper python object to run-operation AFAIK so this is an easy workaround
tests/functional/adapter/list_relations_tests/test_special_characters.py
Outdated
Show resolved
Hide resolved
* revert change for quoting policy issue * make revet in both places * update format to ignore identifier name * update test to use SnowflakeRelation and render out the schema_relation, update macros to have condtion to check based on type * remove unneeded comments from updating test * add test based on quoting schema names * update name to more generic my_model (cherry picked from commit 57285d5)
* quoting config not working with 1.8 (#1075) * revert change for quoting policy issue * make revet in both places * update format to ignore identifier name * update test to use SnowflakeRelation and render out the schema_relation, update macros to have condtion to check based on type * remove unneeded comments from updating test * add test based on quoting schema names * update name to more generic my_model (cherry picked from commit 57285d5) * local pre-commit run --------- Co-authored-by: Matthew McKnight <[email protected]> Co-authored-by: McKnight-42 <[email protected]>
resolves #1074
docs dbt-labs/docs.getdbt.com/#
Problem
During decoupling work we updated some naming conventions in some macros that seem to go around quotiing_policy, reverting to put us back in good standing.
Solution
revert to using
schema_relation
where able in theadapters.sql
if schema_relation is a string, otherwise useschema_relation.include(indentifier=False)
tests/functional/adapter/list_relations_tests/test_pagination.py
update tests in
tests/functional/adapter/list_relations_tests/test_pagination.py
to use aSnowflakeRelation
object instead of adict
add a new test based of the edgecase from issue.
Todo:
Checklist