-
Notifications
You must be signed in to change notification settings - Fork 124
Add Glean Health Scorecard sql-generators #8398
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
base: main
Are you sure you want to change the base?
Conversation
This is an attempt to add sql-generators for the Glean health scorecard derived tables and views.
This comment has been minimized.
This comment has been minimized.
BigQuery infers NULLABLE mode for fields in UNION ALL queries.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
…eed up dry-run validation
This comment has been minimized.
This comment has been minimized.
|
@travis79 I moved templates to a new generator and modified them to produce cross-app combined tables. |
Thank you very much for the assistance! |
This comment has been minimized.
This comment has been minimized.
| @@ -0,0 +1,74 @@ | |||
| -- Query for telemetry health glean errors across all applications | |||
| {% for app in applications %} | |||
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.
I would recommend calling this app_id to disambiguate with app_name. This has happened in other generators and made things more confusing than they should be
This comment has been minimized.
This comment has been minimized.
… into health-scorecard-sql-generators
Integration report for "Merge remote-tracking branch 'origin/health-scorecard-sql-generators' into health-scorecard-sql-generators"
|
Description
Add sql-generators for the Glean health scorecard derived tables and views.
Related Tickets & Documents
Reviewer, please follow this checklist