Closed Bug 1909246 Opened 1 year ago Closed 1 year ago

Up the general global dispatcher limit from 1000 to more

Categories

(Data Platform and Tools :: Glean: SDK, task, P1)

task

Tracking

(Not tracked)

RESOLVED FIXED

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.

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.

Assignee: nobody → tlong
Priority: P3 → P1

fwiw I have this query, that looks at the overflow counts by some (arbitrary) buckets: https://sql.telemetry.mozilla.org/queries/101627/source#250394

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.

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: