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-780: Upgrade dbt-core version and dbt-impala adapater version to 1.7 #197

Merged
merged 5 commits into from
May 4, 2024

Conversation

vamshikolanu
Copy link
Collaborator

@vamshikolanu vamshikolanu commented May 2, 2024

Describe your changes

Upgrade dbt-core version and dbt-hive adapater version to 1.7

dbt-core has removed hologram dependency as part of this PR: https://github.com/dbt-labs/dbt-core/pull/8437/files
Hence removing it from dbt-impala too.

Internal Jira ticket number or external issue link

https://jira.cloudera.com/browse/DBT-780

Testing procedure/screenshots(if appropriate):

https://gist.github.com/vamshikolanu/f6e0c89136d34810b5dfb2b3874f956f

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have formatted my added/modified code to follow pep-8 standards
  • I have checked suggestions from python linter to make sure code is of good quality.

@vamshikolanu vamshikolanu changed the title upgrade_to_v1.7 Upgrade dbt-core version and dbt-impala adapater version to 1.7 May 2, 2024
@vamshikolanu vamshikolanu changed the title Upgrade dbt-core version and dbt-impala adapater version to 1.7 DBT-780: Upgrade dbt-core version and dbt-impala adapater version to 1.7 May 2, 2024
@niteshy
Copy link
Collaborator

niteshy commented May 3, 2024

please update the test plan and doc requirement too eg cloudera/dbt-hive#150

@vamshikolanu vamshikolanu merged commit a3aec0b into master May 4, 2024
@niteshy niteshy deleted the upgrade_to_v1.7 branch May 17, 2024 05:34
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.

2 participants