Skip to content

add new accounts db count table. remove old one#1116

Merged
sean-rose merged 6 commits into
mozilla:mainfrom
clouserw:20241205-accounts-tables
Dec 6, 2024
Merged

add new accounts db count table. remove old one#1116
sean-rose merged 6 commits into
mozilla:mainfrom
clouserw:20241205-accounts-tables

Conversation

@clouserw
Copy link
Copy Markdown
Member

@clouserw clouserw commented Dec 5, 2024

See related new table @ mozilla/bigquery-etl#6511 . /cc @akkomar @sean-rose

Comment thread custom-namespaces.yaml Outdated
Comment thread custom-namespaces.yaml Outdated
Comment thread custom-namespaces.yaml Outdated
Comment thread custom-namespaces.yaml Outdated
Comment thread custom-namespaces.yaml
Comment on lines +620 to +623
measures:
count_sum:
type: sum
sql: ${count}
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 kind of an unrelated change, but we'll allow it. 😉

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

😰

clouserw and others added 4 commits December 5, 2024 15:50
Co-authored-by: Sean Rose <1994030+sean-rose@users.noreply.github.com>
Co-authored-by: Sean Rose <1994030+sean-rose@users.noreply.github.com>
Co-authored-by: Sean Rose <1994030+sean-rose@users.noreply.github.com>
Co-authored-by: Sean Rose <1994030+sean-rose@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@sean-rose sean-rose left a comment

Choose a reason for hiding this comment

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

The moz-fx-data-shared-prod.accounts_backend.monitoring_db_counts view added in mozilla/bigquery-etl#6599 has been deployed, so this is good to go.

@clouserw
Copy link
Copy Markdown
Member Author

clouserw commented Dec 6, 2024

Thanks. I don't have write-access to this repository fwiw

@sean-rose sean-rose merged commit 43c53e1 into mozilla:main Dec 6, 2024
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.

2 participants