Skip to content

Bug 1524670 firefox_nondesktop_exact_mau28#11

Merged
jklukas merged 7 commits intomasterfrom
nondesktop_exact_mau28
Mar 8, 2019
Merged

Bug 1524670 firefox_nondesktop_exact_mau28#11
jklukas merged 7 commits intomasterfrom
nondesktop_exact_mau28

Conversation

@jklukas
Copy link
Copy Markdown
Contributor

@jklukas jklukas commented Mar 7, 2019

In analogy to the firefox_desktop_exact_mau28 tables,
these allow very simple SUM queries that minimize the chance that your
results differ from the calculations used for 2019 goals.

@jklukas jklukas requested a review from relud March 7, 2019 16:40
@mreid-moz
Copy link
Copy Markdown
Contributor

Does this cover Bug 1524670?

@jklukas
Copy link
Copy Markdown
Contributor Author

jklukas commented Mar 7, 2019

Does this cover Bug 1524670?

Sure does.

@jklukas jklukas changed the title nondesktop_exact_mau28 Bug 1524670 nondesktop_exact_mau28 Mar 7, 2019
Copy link
Copy Markdown
Contributor

@relud relud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm if we change the name of the tier1 columns, everything else is optional

Comment thread sql/nondesktop_exact_mau28_by_dimensions_v1.sql Outdated
Comment thread sql/nondesktop_exact_mau28_by_dimensions_v1.sql Outdated
Comment thread sql/nondesktop_exact_mau28_by_dimensions_v1.sql Outdated
SELECT
*,
DATE_DIFF(submission_date, date_last_seen, DAY) AS _inactive_days,
DATE_DIFF(submission_date, date_last_seen_in_tier1_country, DAY) AS _inactive_days_tier1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is an excellent optimization that I want to add along with wau to the desktop table

Comment thread sql/nondesktop_exact_mau28_v1.sql Outdated
Comment thread sql/nondesktop_exact_mau28_by_dimensions_v1.sql
Comment thread sql/nondesktop_exact_mau28_by_dimensions_v1.sql Outdated
Comment thread sql/nondesktop_exact_mau28_by_dimensions_v1.sql
@jklukas jklukas force-pushed the nondesktop_exact_mau28 branch from 20e2a45 to bd82ca5 Compare March 7, 2019 22:26
@relud
Copy link
Copy Markdown
Contributor

relud commented Mar 7, 2019

the CI failure is due to multiple test suites running at the same time and getting a naming collision, working on a fix

Copy link
Copy Markdown
Contributor

@relud relud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

after having some things explained to me: lgtm as-is or with a firefox_ prefix

jklukas added 7 commits March 7, 2019 19:07
In analogy to the `firefox_desktop_exact_mau28` tables,
these allow very simple SUM queries that minimize the chance that your
results differ from the calculations used for 2019 goals.
@jklukas jklukas force-pushed the nondesktop_exact_mau28 branch from 2d3ecc1 to 998e285 Compare March 8, 2019 00:07
@jklukas jklukas changed the title Bug 1524670 nondesktop_exact_mau28 Bug 1524670 firefox_nondesktop_exact_mau28 Mar 8, 2019
@jklukas jklukas merged commit e7691d3 into master Mar 8, 2019
@jklukas jklukas deleted the nondesktop_exact_mau28 branch March 8, 2019 00:13
quiiver pushed a commit that referenced this pull request Jun 25, 2024
Update workspace attach location
quiiver pushed a commit that referenced this pull request Jun 25, 2024
Import etl-graph as subtree instead of git module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants