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

fix: dbt-athena types mapping for complex types #8264

Merged
merged 3 commits into from
Jun 22, 2023
Merged

fix: dbt-athena types mapping for complex types #8264

merged 3 commits into from
Jun 22, 2023

Conversation

svdimchenko
Copy link
Contributor

Problem statement

I get the following error for complex data types for dbt-athena adapter:

  File "/usr/local/lib/python3.10/site-packages/datahub/ingestion/source/sql/sql_types.py", line 245, in resolve_athena_modified_type
    return ATHENA_SQL_TYPES_MAP[type_string]
KeyError: 'array<string>'

Current PR is fixing the issue.

Checklist

@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Jun 20, 2023
Copy link
Collaborator

@hsheth2 hsheth2 left a comment

Choose a reason for hiding this comment

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

LGTM

@treff7es treff7es merged commit 5b9fd97 into datahub-project:master Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ingestion PR or Issue related to the ingestion of metadata
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants