Closed Bug 1626403 Opened 6 years ago Closed 6 years ago

Python: Do ping uploading in a separate process

Categories

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

Product:
Data Platform and Tools
Component:
Glean: SDK
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mdroettboom, Assigned: mdroettboom)

References

Details

The Python bindings now do most Glean work (and I/O) in a separate thread.

However, we should move the ping uploading to its own process. This would allow it to continue to run even after the main application has shut down (which could be some time in the event of a network outage or degradation). As this is by far the slowest and least reliable thing Glean does, it will also make the atexit timeout implemented in bug 1626086 less critical.

Not every environment may play well with forking/multiprocessing (e.g. Windows GUI applications packaged using PyInstaller, like mozregression-gui), so I'd be cautious about making this required.

Assignee: nobody → mdroettboom
Priority: P3 → P1
Whiteboard: [telemetry:glean-rs:m?]

Fixed by https://github.com/mozilla/glean/pull/798 (which also makes this feature opt-out-able)

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