Closed Bug 1505395 Opened 6 years ago Closed 6 years ago

Add support for gathering telemetry from other libraries

Categories

(Toolkit :: Telemetry, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
Tracking Status
firefox65 --- affected

People

(Reporter: Dexter, Assigned: mdroettboom)

References

Details

(Whiteboard: [telemetry:mobilesdk:m5])

Attachments

(1 file)

Other libraries in android-components might want to push their own telemetry to glean, having it take care of the assembly and the delivery of pings. This bug is about investigating this possibility, kicking off a document to gather the requirements
Blocks: 1491345
Priority: -- → P3
Whiteboard: [telemetry:mobilesdk:m4]
Depends on: 1523307
Depends on: 1523331
No longer depends on: 1523307
Depends on: 1523307
Assignee: nobody → mdroettboom

:tlong made a good point elsewhere: Won't the libraries have to either try and initialize, or wait for initialization before they can do anything with Glean? I know that the init function throws an IllegalStateException but is that going to be an issue for a consuming application/library? I know that it can be handled with a try but that seems like a harsh way to indicate that the library is already initialized. Also, what happens if a library tries to initialize with a different configuration?

I would summarize this as: Make sure initialization is as straightforward and fool-proof as possible in light of the new multiple-libraries-using-glean-together world.

Depends on: 1523977
Depends on: 1523979
Whiteboard: [telemetry:mobilesdk:m4] → [telemetry:mobilesdk:m5]
Depends on: 1524993
Depends on: 1525333
Attached file GitHub Pull Request (deleted) —
No longer depends on: 1525333
Depends on: 1525333
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Depends on: 1525904
No longer depends on: 1525333
Status: REOPENED → RESOLVED
Closed: 6 years ago6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: