-
Notifications
You must be signed in to change notification settings - Fork 220
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
Data review: Places maintenance metrics #5122
Comments
I would say that this data is Category 1 (technical data) instead of interaction data (see https://wiki.mozilla.org/Data_Collection) |
Yes that seems better, updated the description. |
Data Review Form (to be filled by Data Stewards)
Resultdata-review+ |
The SQLite documentation describes the amount of time the operation should take in rough terms, but there are no hard limits or upper bounds. We believe that it shouldn't negatively impact users, but we can't be sure without metrics.
We couldn't think of any ways to determine if the pruning was successful other than metrics.
Not that we know of.
Measurement Description: Time spent on the runMaintanence()` function.
Data Collection Category: Category 1 “Technical data”
Tracking bug: Rework places maintenance code #5115
Measurement Description: DB size after the
runMaintanence()
function.Data Collection Category: Category 1 “Technical data”
Tracking bug: Rework places maintenance code #5115
This is scoped to a time-limited experiment/project until date 04-01-2023
All channels in all countries in all locales, for Android
Standard Firefox telemetry controls
We will examine a histogram of the time taken. If we see too many examples of extremely long maintenance operations, we will rework the code to run faster and/or be interruptable. We will also examine a histogram of places Db size. If there are many database that are significantly over the target size, we will increase either the pruning frequency or amount visits pruned per run.
The SACI and Android teams
No
┆Issue is synchronized with this Jira Task
The text was updated successfully, but these errors were encountered: