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

some models for dbt-impala-example are not appearing in Hue when adapted to run with dbt-spark-livy adapter #8

Merged
merged 1 commit into from
Jul 1, 2022

Conversation

tovganesh
Copy link
Collaborator

Internal Ticket: https://jira.cloudera.com/projects/DBT/issues/DBT-124

Synopsis: When debugging the above ticket, the dbt logs indicated that there was an error in list_relations_without_caching indicating a NoneType was passed to the macro.
Further it was observed that when an actual error by SQL executing in the warehouse was thrown, the error was silently ignored.
This PR addresses this issue.

Testplan:
Clone and successfully execute dbt seed and dbt run for the project https://github.com/TapasSenapati/dbt-spark-example

@tovganesh tovganesh marked this pull request as ready for review June 30, 2022 15:11
…ted to run with dbt-spark-livy adapter

Internal Ticket: https://jira.cloudera.com/projects/DBT/issues/DBT-124

Synopsis: When debuging the above ticket, the dbt logs indicated that there was an error in list_relations_without_caching indicating a NoneType was passed to the macro.
Further it was observed that when an actual error by SQL executing in the warehouse was thrown, the error was siliently ignored.
This PR addresses this issue.

Testplan:
Clone and sucessfully execute dbt seed and dbt run for the project https://github.com/TapasSenapati/dbt-spark-example
@TapasSenapati
Copy link
Collaborator

LGTM if we have tested that json object is logged properly in case of an actual error/exception.

Also might need to figure other places at which this could happen. But that is for a separate day and ticket.

@TapasSenapati TapasSenapati reopened this Jun 30, 2022
@TapasSenapati TapasSenapati merged commit 4fb964b into main Jul 1, 2022
@tovganesh tovganesh deleted the dbt_124_fix_a branch October 14, 2022 05:05
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