Skip to content

dbt-core 1.10 support changes - #570

Merged
koletzilla merged 7 commits into
mainfrom
ClickHouse/test-with-1.10-dbt-core-version
Nov 28, 2025
Merged

koletzilla merged 7 commits into
mainfrom
ClickHouse/test-with-1.10-dbt-core-version

Conversation

@koletzilla

@koletzilla koletzilla commented Nov 21, 2025 •

Copy link
Copy Markdown
Contributor

Closes #490

  • Change tests to use dbt-core 1.10 to start validating new functionality.
  • Validate that the new --sample flag (docs) works. Add tests to cover the implementation.

@koletzilla
koletzilla marked this pull request as ready for review November 26, 2025 11:59

@windsurf-bot windsurf-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 To request another review, post a new comment with "/windsurf-review".

Comment thread dev_requirements.txt
Comment on lines -1 to -4
dbt-core>=1.9.0,<1.10
dbt-core>=1.10.0,<1.11
dbt-tests-adapter>=1.10,<2.0
clickhouse-connect>=0.7.6
clickhouse-driver>=0.2.7

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • clickhouse-connect and clickhouse-driver are already defined in setup.py
  • dbt-core updated to 1.10 so test can run against it.

check_table_does_not_exist(project.adapter, "invalid")
check_table_does_not_exist(project.adapter, "skip")
assert "PASS=5 WARN=0 ERROR=1 SKIP=1 TOTAL=7" in output
assert "PASS=5 WARN=0 ERROR=1 SKIP=1 NO-OP=0 TOTAL=7" in output

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only this test "broke" after testing with 1.10 :D

@koletzilla koletzilla self-assigned this Nov 26, 2025
@koletzilla
koletzilla merged commit 8dc2cd0 into main Nov 28, 2025
17 checks passed
@koletzilla
koletzilla deleted the ClickHouse/test-with-1.10-dbt-core-version branch December 3, 2025 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[dbt-core v1.10.0] - Verify & Document Sample mode

2 participants