Closed Bug 1499269 (bmo-phab-refactor-1) Opened 6 years ago Closed 6 years ago

Refactor common parts of the feed daemon and improve timeout logging

Categories

(bugzilla.mozilla.org :: Phabricator Integration, enhancement)

Product:
bugzilla.mozilla.org
Component:
Phabricator Integration
Version:
Production
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: dylan, Assigned: dylan)

References

Details

Attachments

(1 file)

GitHub Pull Request
45 bytes, text/x-github-pull-request
Details | Review
There's a bit of duplicated code in the feed daemon which should be refactored. This allows to make the timeout log message mention which query timed out. Also added logging for any errors raised by _cleanup() and made that exit 1.
Attached file GitHub Pull Request —
Merged to master.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Alias: bmo-phab-refactor-1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: