1093871 - Telemetry probe for number of items in reading list

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Reading List, defect)

Description

[:Margaret Leibovic]

Something similar to: http://mxr.mozilla.org/mozilla-central/search?string=FENNEC_THUMBNAILS_COUNT

Comment 1

[:Margaret Leibovic]

Attachment: Telemetry probe for number of items in reading list

I'm uncertain about the histogram values. What do you think? Will people have more than 500 reading list items?

Comment 2

[Richard Newman [:rnewman]]

Comment on attachment 8519101 [details] [diff] [review]
Telemetry probe for number of items in reading list

Review of attachment 8519101 [details] [diff] [review]:
-----------------------------------------------------------------

Perhaps bump 500 to 1000; this is an exponential histogram, so we can easily accommodate, and it's nice to know what our really top-end users will eventually hit.

Bump the number of buckets according to how much granularity you want at the low end. If we care about 0-1, 1-3, 4-10, then we might want more than ten buckets. Try it and see how you feel.

Comment 3

[:Margaret Leibovic]

(In reply to Richard Newman [:rnewman] from comment #2)

> Perhaps bump 500 to 1000; this is an exponential histogram, so we can easily
> accommodate, and it's nice to know what our really top-end users will
> eventually hit.

Okay, 1000 sounds good. I would be *really* surprised if a user has over 500 items in their reading list, but users do surprising things sometimes :)

> Bump the number of buckets according to how much granularity you want at the
> low end. If we care about 0-1, 1-3, 4-10, then we might want more than ten
> buckets. Try it and see how you feel.

I don't think we really care about too much granularity. I mostly want to inform our decisions about how to manage the cache, particularly how to fetch content in the background after items are added to the list.

Comment 4

[:Margaret Leibovic]

https://hg.mozilla.org/integration/fx-team/rev/986d09aa7749

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/986d09aa7749

Comment 6

[:Margaret Leibovic]

Comment on attachment 8519101 [details] [diff] [review]
Telemetry probe for number of items in reading list

Approval Request Comment
[Feature/regressing bug #]: none, but we're working on reading list improvements for Fx36, so this will help inform our decisions
[User impact if declined]: we can't tell how much people use the reading list feature
[Describe test coverage new/current, TBPL]: tested locally, landed on m-c
[Risks and why]: low-risk, adds a telemetry probe
[String/UUID change made/needed]: none

Comment 7

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8519101 [details] [diff] [review]
Telemetry probe for number of items in reading list

Low risk addition of new Telemetry probe.

Beta+
Aurora+

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/a8221dcf14d9
https://hg.mozilla.org/releases/mozilla-beta/rev/f85f63d11f68