Skip to content

Commit

Permalink
Merge pull request #1719 from elementary-data/ele-3695-fix-elementary…
Browse files Browse the repository at this point in the history
…_unique_id

handle case where column name is an empty string
  • Loading branch information
ofek1weiss authored Oct 9, 2024
2 parents 0f4a215 + 982c5ce commit 60fd215
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
when elementary_test_results.test_type = 'schema_change' then elementary_test_results.test_unique_id
{# In old versions of elementary, elementary_test_results doesn't contain test_short_name, so we use dbt_test short_name. #}
when dbt_tests.short_name = 'dimension_anomalies' then elementary_test_results.test_unique_id
else coalesce(elementary_test_results.test_unique_id, 'None') || '.' || coalesce(elementary_test_results.column_name, 'None') || '.' || coalesce(elementary_test_results.test_sub_type, 'None')
else coalesce(elementary_test_results.test_unique_id, 'None') || '.' || coalesce(nullif(elementary_test_results.column_name, ''), 'None') || '.' || coalesce(elementary_test_results.test_sub_type, 'None')
end as elementary_unique_id,
elementary_test_results.invocation_id,
elementary_test_results.data_issue_id,
Expand Down

0 comments on commit 60fd215

Please sign in to comment.