Page MenuHomePhabricator

Allow TransactionEditor to move publishing work to the daemons
ClosedPublic

Authored by epriestley on Jun 2 2015, 12:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 26, 3:08 PM
Unknown Object (File)
Fri, Mar 22, 9:10 PM
Unknown Object (File)
Fri, Mar 22, 8:20 PM
Unknown Object (File)
Fri, Mar 22, 8:20 PM
Unknown Object (File)
Fri, Mar 22, 8:20 PM
Unknown Object (File)
Tue, Mar 5, 2:37 PM
Unknown Object (File)
Tue, Mar 5, 9:13 AM
Unknown Object (File)
Tue, Mar 5, 8:59 AM
Subscribers

Details

Summary

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

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
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.