-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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(ingest/snowflake): fix incorrect tag urn case, improve tag display name #7758
fix(ingest/snowflake): fix incorrect tag urn case, improve tag display name #7758
Conversation
TagAssociation( | ||
tag=make_tag_urn(self.snowflake_identifier(tag.identifier())) | ||
) | ||
for tag in table.tags |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was generating mixed case URNs as opposed to lowercasing the urn, as is done at other places where urns for snowflake tags and other entities are generated.
@@ -35,6 +35,9 @@ create user datahub_user display_name = 'DataHub' password='' default_role = dat | |||
|
|||
// Grant the datahub_role to the new DataHub user. | |||
grant role datahub_role to user datahub_user; | |||
|
|||
// Optional - required if extracting lineage, usage or tags (without lineage) | |||
grant imported privileges on database snowflake to role datahub_role; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is added here so that its present in the block containing other permissions. Earlier this was only present in the later sections after a lot of text and it was observed that this permission gets missed leading to missing lineage after ingestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks - this is super helpful
Checklist