-
Notifications
You must be signed in to change notification settings - Fork 104
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat(DENG-6890): Update mobile kpi support metric aggregates to suppo…
…rt dma analysis (#6783) * feat: add device_type and device_manufacturer to KPI support metrics datasets * feat: ensure all mobile kpi support metric aggregated dataset have device_type and device_manufacturer dimensions * feat: add device_type and device_manfacturer to the mobile kpi support metric aggregate queries * feat: tweak active_users view device_type value setting logic * feat: lower case device_manufacturer for some cases where the same manufacturer value has different casing * feat: bucket device_manufacturers into "other" bucket in case when they have a lower occurance to prevent from exploding the aggregate tables * fix: retention_clients some selection fields being ambiguous * feat: Use dense rank instead of count for bucketing all device_manufacturers into other bucket if they are rank 151 or more * fix: using incorrect source table for ranking device_manufacturers * feat: make sure that device_manufacturer is lowered prior to joining
- Loading branch information
Showing
10 changed files
with
138 additions
and
2 deletions.
There are no files selected for viewing
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
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
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
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
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
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
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
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
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
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
b81a7a0
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 "feat(DENG-6890): Update mobile kpi support metric aggregates to support dma analysis (#6783)"
sql.diff
Click to expand!
Link to full diff