Up the general global dispatcher limit from 1000 to more
Categories
(Data Platform and Tools :: Glean: SDK, task, P1)
Tracking
(Not tracked)
People
(Reporter: janerik, Assigned: travis_)
Details
(Whiteboard: [telemetry:glean-rs:m?])
Attachments
(1 file)
Right now we have a queue limit of 1000 in general, but up that limit to 1000000 when Gecko is built for Desktop.
We should probably just remove that feature toggle and have the general higher limit and ship it on Android too.
Maybe we should have a look at how often we hit that limit on Android.
| Assignee | ||
Comment 1•1 year ago
|
||
Looks like we would definitely benefit from a larger pre-init queue in Fenix: https://sql.telemetry.mozilla.org/queries/101646/source#250446. The big spike you see is most likely related to the noisy Nimbus event which was rapid firing before Glean gets initialized back in May.
I'll have a patch inbound to remove the conditional compilation in favor of the 1M queue size for the default.
Comment 2•1 year ago
|
||
| Reporter | ||
Comment 3•1 year ago
|
||
fwiw I have this query, that looks at the overflow counts by some (arbitrary) buckets: https://sql.telemetry.mozilla.org/queries/101627/source#250394
| Assignee | ||
Comment 4•1 year ago
|
||
I'll revisit these queries once this makes it into Fenix (and Firefox iOS) to ensure everything is working as intended and file new bugs as needed. I'm closing this one as FIXED.
Description
•