-
Notifications
You must be signed in to change notification settings - Fork 37
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
Expose environment variables for database and schema names for tag setting #28
Comments
Yes. we do like to have a centralized place for all tags. It would be much easier to associate centralized data masking policy with tags. |
Yes, please, let us use tags defined in other dbs/schemas. In particular we might want to use dedicated snowflake tags SNOWFLAKE.CORE.SEMANTIC_CATEGORY and/or SNOWFLAKE.CORE.PRIVACY_CATEGORY. |
Current behaviour is error prone. I'd like to allow our users to use tags from our predefined set only (we manage tags and other Snowflake objects with terraform). If they make a typo a new tag will be created and they will never now, and my monitoring will become useless. So I would like to be able to switch off the auto-creation. |
I've copied this one into a new issue |
Yep we wrote our own macro for tags, for the same reason..we managed the tag list centrally and didn't want tags being auto-created. We also had a separate central Snowflake database just for data governance (including data masking policies and tags), so needed the tags to reference that from other databases. |
Description:
Expose two environment variables to allow package users to specify where tags should be created in Snowflake when leveraging the apply_meta_as_tags macro.
The current behaviour is to use the model's database and schema, which should work well for most but may not be ideal for all circumstances. This behaviour was implemented thanks to a contribution from @calleo in PR #25 .
If this issue is relevant to you, please leave a comment or thumbs up reaction to help us prioritize!
The text was updated successfully, but these errors were encountered: