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
feat: add is_init() to glean_usage queries #5509
base: main
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
SELECT | ||
* | ||
FROM | ||
`{{ project_id }}.{{ app_name }}.metrics_clients_daily` AS m | ||
WHERE | ||
FALSE |
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.
We need to get the right metrics fields here:
SELECT | |
* | |
FROM | |
`{{ project_id }}.{{ app_name }}.metrics_clients_daily` AS m | |
WHERE | |
FALSE | |
SELECT | |
DATE(@submission_date) AS submission_date, | |
_current.client_id, | |
_current.sample_id, | |
_current.normalized_channel, | |
_current.n_metrics_ping, | |
_current.days_sent_metrics_ping_bits AS days_sent_metrics_ping_bits, | |
{% if app_name in metrics -%} | |
{% for metric in metrics[app_name] -%} | |
_current.{{metric}} AS {{metric}}, | |
{% endfor -%} | |
{% endif -%} | |
FROM | |
`{{ project_id }}.{{ app_name }}.metrics_clients_daily` AS _current | |
WHERE FALSE |
Integration report for "Merge branch 'main' into feat/add_is_init-to-glean_usage-queries"
|
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.
The changes to this view and some of the changes to clients_last_seen_joined.query.sql
seem like they're unrelated to the purpose of adding is_init()
, and are kind of bleeding over from #5434. I'd suggest removing these unrelated changes.
{% raw %} | ||
{% if is_init() %} | ||
{% endraw %} |
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.
For consistency (e.g. you specifically changed baseline_clients_last_seen_v1.query.sql
to switch to this single-line format).
{% raw %} | |
{% if is_init() %} | |
{% endraw %} | |
{% raw %}{% if is_init() %}{% endraw %} |
(ditto for the {% else %}
and {% endif %}
parts)
Checklist for reviewer:
<username>:<branch>
of the fork as parameter. The parameter will also show upin the logs of the
manual-trigger-required-for-fork
CI task together with more detailed instructions.For modifications to schemas in restricted namespaces (see
CODEOWNERS
):┆Issue is synchronized with this Jira Task