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

Tableau: snowflake columns should not blindly lowercased if ingest_tables_external is set to true #9847

Closed
alexs-101 opened this issue Feb 14, 2024 · 1 comment · Fixed by #9838
Labels
bug Bug report stale

Comments

@alexs-101
Copy link
Contributor

The tableau plugin has the snowflake related crutch in the function

def get_upstream_columns_of_fields_in_datasource(

that blindly lowercasing snowflake table column names

If we ingest metedata from Tableau using ingest_tables_external: true, so the snowflake tables schema comes from Tableau metadata, the linage is not shown in UI due to case sensitivity of the linage links. Thanx to the blindly applied snowflake crutch lowercasing column names.

If ingest_tables_external is enabled, we should not lowercase column names

The fix is by the link
#9838

Copy link

This issue is stale because it has been open for 30 days with no activity. If you believe this is still an issue on the latest DataHub release please leave a comment with the version that you tested it with. If this is a question/discussion please head to https://slack.datahubproject.io. For feature requests please use https://feature-requests.datahubproject.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report stale
Projects
None yet
1 participant