Page MenuHomePhabricator

Allow the Trigger daemon to hibernate, reducing processes to 0
ClosedPublic

Authored by epriestley on Feb 24 2017, 5:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 6:28 PM
Unknown Object (File)
Fri, Dec 20, 11:49 AM
Unknown Object (File)
Thu, Dec 19, 3:26 AM
Unknown Object (File)
Wed, Dec 18, 2:20 PM
Unknown Object (File)
Wed, Dec 18, 2:20 PM
Unknown Object (File)
Wed, Dec 18, 2:20 PM
Unknown Object (File)
Wed, Dec 18, 12:37 PM
Unknown Object (File)
Thu, Dec 12, 12:40 AM
Subscribers
None

Details

Summary

Ref T12298. The trigger daemon already has routine long-term sleep, and few external events can impact when it should ideally wake up. The relevant events are:

  • Someone creates a new Nuance source (ideally, we should wake up right away and start polling it).
  • Someone creates a Calendar event about 16 minutes in the future (ideally, we should send them a reminder in about a minute).
  • Someone changes GC config to be extremely aggressive (ideally, we should immediately respect the change).

None of these cases are very important. We don't hibernate for more than 3 minutes, so the worst case is that your Nuance source takes 3 minutes to start importing or your Calendar notification comes two minutes too late (13 minutes before the event instead of 15).

This change makes GC sightly more CPU-expensive on average: currently, we do a GC sweep every 4 hours. After this change, we'll end up doing one every 3 minutes, because we lose the fact that we did a sweep recently when the daemon restarts.

We could fix this by keeping track of when the last GC sweep was in the database, instead of in the Daemon process, but the cost of a sweep is normally very small so I don't plan to do this anytime soon.

Test Plan
  • Ran bin/phd debug trigger, saw daemon go through 3-minute hibernate + restart cycles.
  • Ran bin/phd debug task, saw daemon run normally.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Feb 24 2017, 5:36 PM
This revision was automatically updated to reflect the committed changes.