-
Notifications
You must be signed in to change notification settings - Fork 101
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
add views for desktop_new_profiles clients and aggregates (#6258)
* add views for desktop_new_profiles clients and aggregates * move fiels from desktop_new_profiles_aggregates to desktop_new_profiles * fix name of view by adding clients to it * Update sql/moz-fx-data-shared-prod/telemetry/desktop_new_profiles_clients/view.sql Co-authored-by: Chelsey Beck <[email protected]> --------- Co-authored-by: Chelsey Beck <[email protected]>
- Loading branch information
1 parent
f826580
commit 9632554
Showing
2 changed files
with
31 additions
and
0 deletions.
There are no files selected for viewing
7 changes: 7 additions & 0 deletions
7
sql/moz-fx-data-shared-prod/telemetry/desktop_new_profiles/view.sql
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
CREATE OR REPLACE VIEW | ||
`moz-fx-data-shared-prod.telemetry.desktop_new_profiles` | ||
AS | ||
SELECT | ||
* | ||
FROM | ||
`moz-fx-data-shared-prod.telemetry_derived.desktop_new_profiles_aggregates_v1` |
24 changes: 24 additions & 0 deletions
24
sql/moz-fx-data-shared-prod/telemetry/desktop_new_profiles_clients/view.sql
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
CREATE OR REPLACE VIEW | ||
`moz-fx-data-shared-prod.telemetry.desktop_new_profiles_clients` | ||
AS | ||
SELECT | ||
a.*, | ||
`moz-fx-data-shared-prod`.udf.funnel_derived_clients( | ||
normalized_os, | ||
first_seen_date, | ||
app_build_id, | ||
attribution_source, | ||
attribution_ua, | ||
startup_profile_selection_reason, | ||
distribution_id | ||
) AS funnel_derived, | ||
`moz-fx-data-shared-prod`.udf.distribution_model_clients(distribution_id) AS distribution_model, | ||
`moz-fx-data-shared-prod`.udf.partner_org_clients(distribution_id) AS partner_org, | ||
IF( | ||
LOWER(IFNULL(isp_name, '')) <> "browserstack" | ||
AND LOWER(IFNULL(distribution_id, '')) <> "mozillaonline", | ||
TRUE, | ||
FALSE | ||
) AS is_desktop | ||
FROM | ||
`moz-fx-data-shared-prod.telemetry_derived.clients_first_seen_v3` AS a |
9632554
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.
Integration report for "add views for desktop_new_profiles clients and aggregates (#6258)"
sql.diff
Click to expand!
Link to full diff