Page MenuHomePhabricator

Allow TransactionEditor to move publishing work to the daemons

Authored by epriestley on Jun 2 2015, 12:57 AM.



Ref T6367. This is similar to D11329, but not quite as ambitious.

Allow Editors to implement supportsWorkers() and move their publishing work into a daemon. So far, only Paste supports this.

Most of the complexity here is saving and restoring state across the barrier between the web process and the worker process, but I think this is ~90% of it and then we'll pick up a couple of random things in applications.

I'm primarily trying to keep this as gradual as possible.

Test Plan
  • Published transactions with and without daemon support.
  • Looked at mail, feed.

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Allow TransactionEditor to move publishing work to the daemons.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
  • Docs + minor signature fix.
  • Remove now-unused 'editorClass'.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 2 2015, 4:50 PM
This revision was automatically updated to reflect the committed changes.