This change has been marked as submitted.

Describe the submission (optional):

Merged (slightly adjusted) into master

Summary:

[Spwn] add background job and job dispatcher interfaces

Review Request #24 — Created Dec. 12, 2012 and submitted 12 years, 4 months ago

Information
Owner:	olivier*
Repository:	Lunr
Branch:	https://github.com/owizen/lunr/tree/bgjob
Bugs:	9434
Depends On:
Reviewers
Groups:	lunr
People:

Description

Add the interfaces to use in the background job architecture

Testing Done

Issues

Description	From	Last Updated
nitpick, I'm not sure Core is the right place for it. Might need a new one. Know a good one? ...	pprkut	12 years, 4 months ago
we did not start this in 2010 ;)	pprkut	12 years, 4 months ago
that doesn't match the Resque expectations, as it needs a perform() method. Also setUp() and tearDown() are missing	pprkut	12 years, 4 months ago
wrong package	pprkut	12 years, 4 months ago
superfluous line	pprkut	12 years, 4 months ago
wrong package	pprkut	12 years, 4 months ago
There are no open issues

system/libraries/core/interface.job.inc.php (Diff revision 1)

The issue has been resolved. Show all issues

nitpick, I'm not sure Core is the right place for it. Might need a new one. Know a good one? Something that matches the style of Sprk and Tcks?

system/libraries/core/interface.job.inc.php (Diff revision 1)

The issue has been resolved. Show all issues

that doesn't match the Resque expectations, as it needs a perform() method. Also setUp() and tearDown() are missing

Rename and add methods in job interface

-	[Core] add background job and job dispatcher interfaces
+	[Spwn] add background job and job dispatcher interfaces

		Add the interfaces to use in the background job architecture
-
-		The use will be
-
-		class daemon implements JobDispatcherInterface
-		{
-
-		public function dispatch($job, $args)
-		{
-		$j = new $job();
-		$j->run($args);
-		}
-
-		}
-
-		class MyJob implements JobInterface
-		{
-		public function run($args)
-		{
-		Resque::enqueue($args['queue'], $args['class'], $args['args']);
-		}
-		}
-
-		Now you can slay me!