Skip to content
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

Dbt 1.8 migration #244

Merged
merged 8 commits into from
Nov 21, 2024
Merged

Dbt 1.8 migration #244

merged 8 commits into from
Nov 21, 2024

Conversation

howareyouman
Copy link
Contributor

@howareyouman howareyouman commented Nov 15, 2024

Summary

dbt-core migration to 1.8.* version

Description

Summarisation of 2 PRs to migrate to dbt-core 1.8.* version

Test Results

I'm expecting it as a baseline for future improvements.
Tests statistics after migration to dbt 1.8 (with CE Dremio on-prem version):
tests:

  • caching: passing
  • component: 2 out of 4 are passing (these tests rely on both DC and Software dremio versions)
  • functional:
    • basic: passing with ignored snapshot tests.
    • dbt_clone: failing
    • dremio_specific: passing
    • empty: failing
    • grants: failing
    • query_comment: passing
    • relation: 1 out of 2 are passing
    • unit_testing: failing
    • utils: passing
  • hooks: 7 out of 12 are passing
  • simple_copy: passing
  • unit: passing

Changelog

  • migration to dbt-core 1.8 is finished
  • some tests are failing still

@howareyouman howareyouman mentioned this pull request Nov 15, 2024
1 task
@howareyouman howareyouman marked this pull request as ready for review November 19, 2024 10:23
README.md Show resolved Hide resolved
dbt/adapters/dremio/api/cursor.py Outdated Show resolved Hide resolved
dbt/adapters/dremio/api/cursor.py Outdated Show resolved Hide resolved
dbt/adapters/dremio/api/rest/endpoints.py Outdated Show resolved Hide resolved
dbt/adapters/dremio/connections.py Show resolved Hide resolved
tests/hooks/test_model_hooks.py Outdated Show resolved Hide resolved
tests/hooks/test_run_hooks.py Outdated Show resolved Hide resolved
@howareyouman howareyouman merged commit 6b65f44 into dremio:main Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants