Moved some of the job processing logging into job and out of worker.

seomoz · Apr 29, 2012 · e001bc8 · e001bc8
1 parent efbab62
commit e001bc8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/qless/job.py b/qless/job.py
@@ -94,7 +94,9 @@ def process(self):
         if method:
             if isinstance(method, types.FunctionType):
                 try:
+                    logger.info('Processing %s in %s' % (self.jid, self.queue))
                     method(self)
+                    logger.info('Completed %s in %s' % (job.jid, queue.name))
                 except Exception as e:
                     # Make error type based on exception type
                     logger.exception('Failed %s in %s: %s' % (self.jid, self.queue, repr(method)))

diff --git a/qless/worker.py b/qless/worker.py
@@ -92,10 +92,8 @@ def work(self):
                     job = queue.pop()
                     if job:
                         seen = True
-                        logger.info('Processing %s in %s' % (job.jid, queue.name))
                         self.setproctitle('Working %s (%s)' % (job.jid, job.klass))
                         job.process()
-                        logger.info('Completed %s in %s' % (job.jid, queue.name))
                         self.clean()
 
                 if not seen: