aboutsummaryrefslogtreecommitdiffstats
path: root/mediagoblin/processing/__init__.py
diff options
context:
space:
mode:
authorSebastian Spaeth <Sebastian@SSpaeth.de>2012-12-20 13:42:37 +0100
committerRodney Ewing <ewing.rj@gmail.com>2013-08-19 14:42:13 -0700
commitbf2dafd1a04ef8050ebf08bb512862a1592998c0 (patch)
tree1732dd80d13a780f0cafffefffd2da3f1854548e /mediagoblin/processing/__init__.py
parentbd0b5daa44691c8cf72ef2ff5a8cdc3b01490722 (diff)
downloadmediagoblin-bf2dafd1a04ef8050ebf08bb512862a1592998c0.tar.lz
mediagoblin-bf2dafd1a04ef8050ebf08bb512862a1592998c0.tar.xz
mediagoblin-bf2dafd1a04ef8050ebf08bb512862a1592998c0.zip
Tweak Celery Task
- Make sure Exceptions are pickleable (not sure if this was not the case but this is the pattern as documented in the celery docs. - Don't create a task_id in the GMG code, but save the one implicitely created by celery. - Don't create a task-id directory per upload. Just store queued uploads in a single directory (this is the most controversial change and might need discussion!!!) Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
Diffstat (limited to 'mediagoblin/processing/__init__.py')
-rw-r--r--mediagoblin/processing/__init__.py7
1 files changed, 4 insertions, 3 deletions
diff --git a/mediagoblin/processing/__init__.py b/mediagoblin/processing/__init__.py
index f3a85940..ae3652cf 100644
--- a/mediagoblin/processing/__init__.py
+++ b/mediagoblin/processing/__init__.py
@@ -181,9 +181,10 @@ class BaseProcessingFail(Exception):
return u"%s:%s" % (
self.__class__.__module__, self.__class__.__name__)
- def __init__(self, **metadata):
- self.metadata = metadata or {}
-
+ def __init__(self, *args, **kwargs):
+ # next line is REQUIRED to have pickable exceptions if you want
+ # to be able to pass in custom arguments (see celery docs)
+ Exception.__init__(self, *args, **metadata)
class BadMediaFail(BaseProcessingFail):
"""