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
support dbt v1.2 #6
Conversation
6d07b6b
to
bc039a1
Compare
5dcd7b2
to
bbad2c1
Compare
f79b23b
to
a8ea83c
Compare
dbt/include/tidb/macros/materializations/incremental/helpers.sql
Outdated
Show resolved
Hide resolved
dbt/include/tidb/macros/materializations/incremental/incremental.sql
Outdated
Show resolved
Hide resolved
resolve the conflicts after #13 merged |
# Conflicts: # .github/workflows/main.yml # README.md # dbt/adapters/tidb/connections.py # dbt/include/tidb/macros/materializations/snapshot/strategies.sql # requirements_dev.txt # setup.py # tests/README.md # tests/conftest.py # tests/functional/adapter/tidb/basic/test_tidb.py # tests/functional/adapter/tidb4_0/basic/test_tidb.py # tests/functional/adapter/tidb5_1/basic/test_tidb.py
@@ -6,26 +6,41 @@ | |||
{%- endfor -%})) | |||
{%- endmacro %} | |||
|
|||
{% macro snapshot_check_all_get_existing_columns(node, target_exists) -%} | |||
{%- set query_columns = get_columns_in_query(node['compiled_sql']) -%} | |||
-- copy from dbt-core v1.2, just alter database=None in adapter.get_relation |
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.
@shiyuhang0 what if a relation object never printed the database? would save you from having to override core's version of this macro?
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.
I think maybe we can delete this override because I found that the database has been set to None in get_relation.
And I will test if it still works after delete the macro
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.
What problem does this PR solve?
close #2
What is changed and how it works?
Check List