Worker - supporting running queued tasks in process
ClosedPublic
Actions

Authored by btrahan on May 8 2014, 8:04 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Apr 29, 2:05 PM

	Unknown Object (File)
	Fri, Apr 26, 5:14 PM

	Unknown Object (File)
	Wed, Apr 24, 9:56 PM

	Unknown Object (File)
	Sun, Apr 21, 4:20 PM

	Unknown Object (File)
	Sat, Apr 13, 5:55 PM

	Unknown Object (File)
	Mon, Apr 8, 11:39 AM

	Unknown Object (File)
	Sun, Apr 7, 5:18 AM

	Unknown Object (File)
	Fri, Apr 5, 12:50 PM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rP61dd5ab6c133: Worker - supporting running queued tasks in process

Summary

Ref D8930. My "send test" for SMS was failing before this patch, and now it works nicely.

Test Plan

Used new code in D8930 that uses $this->queueTask() to get some work done and it got done in process

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan updated this revision to Diff 21412.May 8 2014, 8:04 PM

btrahan retitled this revision from to Worker - supporting running queued tasks in process.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

Herald added subscribers: Korvin, epriestley. · View Herald TranscriptMay 8 2014, 8:04 PM

Harbormaster completed remote builds in B311: Diff 21412.May 8 2014, 8:05 PM

epriestley accepted this revision.May 8 2014, 8:12 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.May 8 2014, 8:12 PM

Closed by commit rP61dd5ab6c133 (authored by @btrahan).

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

daemon/

workers/

PhabricatorWorker.php

4 lines

Diff 21413

View Options

src/infrastructure/daemon/workers/PhabricatorWorker.php

Worker - supporting running queued tasks in processClosedPublicActions