-
Notifications
You must be signed in to change notification settings - Fork 4
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
[SNOW-85] Document RAW tables #94
Conversation
|
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.
🔥 LGTM! Note: I did not carefully read all of the docstrings AND I really hope GPT or copilot was leveraged in this endeavor.
Thanks so much!
Codeium 😄 Thanks! I'll leave it open for @philerooski to give it a final pass jic |
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.
What information did you find in the Confluence documentation that wasn't available in the Synapse-Stack-Builder config?
The descriptions of the tables and columns for |
problem
Most of the bronze layer tables and their columns were missing documentation.
solution
Add table and column comments for all the bronze layer tables that don't have. I used the Synapse-Stack-Builder repo and the Jira documentation for column and table descriptions.
testing
I ran all the scripts against the raw tables in this clone and was able to confirm all table + column comments were applied.