Closed
Bug 1257595
Opened 8 years ago
Closed 8 years ago
profileDate quality issues on Fennec release for "core" pings
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(firefox48 fixed)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: gfritzsche, Assigned: gfritzsche)
References
Details
(Whiteboard: [measurement:client])
Attachments
(1 file)
1.59 KB,
patch
|
mcomella
:
review+
|
Details | Diff | Splinter Review |
The analysis in bug 1255458 [0] found some quality issues with profileDate: * 161 times it is missing * 641 times it is < 10957 (but >= 0) (i.e. a date before the year 2000) * 101 times it is > 17167 (i.e. a date with year 2017+) * Examples of those issues are printed at [54] in the notebook 0: https://gist.github.com/georgf/c9f4824e5d3bbf112db7
Assignee | ||
Comment 1•8 years ago
|
||
I don't have any experience with how much clock breakage we expect on Android expect. Could the wrong dates simply be broken clocks? Or a buggy implementation of us getting the profile age? The 161 missing profileDates surprise me as we haven't documented it as optional.
Comment 2•8 years ago
|
||
We expect lots of insane clocks on Android. Older studies suggest 3.5% of devices will be more than an hour wrong, with around 0.25% being more than a day ahead. http://opensignal.com/reports/timestamps/
(In reply to Georg Fritzsche [:gfritzsche] from comment #1) > The 161 missing profileDates surprise me as we haven't documented it as > optional. Not documenting this is my mistake – the current algo [1] attempts to read times.json and return the value from there. However, if times.json does not exist or the value could not be read, it returns -1 until bug 1246816 is implemented. Do we want to keep this bug open for the instances of clock skew? [1]: https://mxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/telemetry/TelemetryUploadService.java#210
Depends on: 1246816
Flags: needinfo?(gfritzsche)
Assignee | ||
Comment 4•8 years ago
|
||
The clock skew seems to be below the expected level, i can run that down in a bit more detail once we get proper release data. We should note the behavior in the documentation though to avoid surprises.
Flags: needinfo?(gfritzsche)
Assignee | ||
Comment 5•8 years ago
|
||
Note that this will have minor conflicts with the doc changes in bug 1249288, but they should be trivial to resolve.
Attachment #8735844 -
Flags: review?(michael.l.comella)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → gfritzsche
Status: NEW → ASSIGNED
Comment on attachment 8735844 [details] [diff] [review] Document that the 'core' pings profileDate field can be missing Review of attachment 8735844 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me! Note that I already landed bug 1249288. Thanks for keeping the docs up-to-date.
Attachment #8735844 -
Flags: review?(michael.l.comella) → review+
Assignee | ||
Updated•8 years ago
|
Points: --- → 1
Whiteboard: [measurement:client]
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/346ac4ebf250
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•