Closed
Bug 1677441
Opened 4 years ago
Closed 4 years ago
Refactor `dispatcher::launch(.... with_glean(|..|...))` to a single function (RLB)
Categories
(Data Platform and Tools :: Glean: SDK, enhancement, P1)
Data Platform and Tools
Glean: SDK
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Dexter, Assigned: chinyingli.alice, Mentored)
Details
(Whiteboard: [good next bug][lang=Rust])
Attachments
(1 file)
As Jan-Erik rightfully pointed out on github, code such as:
dispatcher::launch(move || crate::with_glean(|glean| metric.add(glean, amount)));
in the rust language bindings could be refactored to a single function like:
launch_with_glean(|glean| ...)
which would internally call the dispatcher and acquire the Glean instance.
This is a potentially good next bug.
Updated•4 years ago
|
Whiteboard: [telemetry:fog:m?] → [telemetry:glean-rs:m?]
Comment 1•4 years ago
|
||
ni?myself to fill in the steps to help
Mentor: jrediger
Flags: needinfo?(jrediger)
Whiteboard: [telemetry:glean-rs:m?] → [telemetry:glean-rs:m?][good next bug][lang=Rust]
Updated•4 years ago
|
Flags: needinfo?(jrediger)
Whiteboard: [telemetry:glean-rs:m?][good next bug][lang=Rust] → [telemetry:glean-rs:backlog][good next bug][lang=Rust]
Updated•4 years ago
|
Priority: P3 → P4
Comment 2•4 years ago
|
||
Updated•4 years ago
|
Assignee: nobody → chinyingli.alice
Priority: P4 → P1
Whiteboard: [telemetry:glean-rs:backlog][good next bug][lang=Rust] → [good next bug][lang=Rust]
Updated•4 years ago
|
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•