19

我有一个运行一些任务的芹菜链。每个任务都可能失败并重试。请参阅下面的快速示例：

from celery import task

@task(ignore_result=True)
def add(x, y, fail=True):
    try:
        if fail:
            raise Exception('Ugly exception.')
        print '%d + %d = %d' % (x, y, x+y)
    except Exception as e:
        raise add.retry(args=(x, y, False), exc=e, countdown=10)

@task(ignore_result=True)
def mul(x, y):
    print '%d * %d = %d' % (x, y, x*y)

和链条：

from celery.canvas import chain
chain(add.si(1, 2), mul.si(3, 4)).apply_async()

运行这两个任务（并假设没有失败），你会得到/看到打印：

1 + 2 = 3
3 * 4 = 12

但是，当添加任务第一次失败并在随后的重试调用中成功时，链中的其余任务不会运行，即添加任务失败，链中的所有其他任务都不会运行，几秒钟后，添加任务再次运行并成功，链中的其余任务（在本例中为 mul.si(3, 4)）不运行。

celery 是否提供了一种方法来从失败的任务中继续失败的链？如果不是，那么完成此任务并确保链的任务以指定的顺序运行并且仅在前一个任务成功执行之后（即使该任务重试几次）的最佳方法是什么？

注1：这个问题可以通过做

add.delay(1, 2).get()
mul.delay(3, 4).get()

但我有兴趣了解为什么链不适用于失败的任务。

4

2 回答 2

12

你发现了一个错误:)

在https://github.com/celery/celery/commit/b2b9d922fdaed5571cf685249bdc46f28acacde3中修复 将成为 3.0.4 的一部分。

于 2012-07-25T17:55:10.230 回答
0

我也有兴趣了解为什么链不适用于失败的任务。

我挖掘了一些芹菜代码，到目前为止我发现的是：

实现发生在 app.builtins.py

@shared_task
def add_chain_task(app):
    from celery.canvas import chord, group, maybe_subtask
    _app = app

    class Chain(app.Task):
        app = _app
        name = 'celery.chain'
        accept_magic_kwargs = False

        def prepare_steps(self, args, tasks):
            steps = deque(tasks)
            next_step = prev_task = prev_res = None
            tasks, results = [], []
            i = 0
            while steps:
                # First task get partial args from chain.
                task = maybe_subtask(steps.popleft())
                task = task.clone() if i else task.clone(args)
                i += 1
                tid = task.options.get('task_id')
                if tid is None:
                    tid = task.options['task_id'] = uuid()
                res = task.type.AsyncResult(tid)

                # automatically upgrade group(..) | s to chord(group, s)
                if isinstance(task, group):
                    try:
                        next_step = steps.popleft()
                    except IndexError:
                        next_step = None
                if next_step is not None:
                    task = chord(task, body=next_step, task_id=tid)
                if prev_task:
                    # link previous task to this task.
                    prev_task.link(task)
                    # set the results parent attribute.
                    res.parent = prev_res

                results.append(res)
                tasks.append(task)
                prev_task, prev_res = task, res

            return tasks, results

        def apply_async(self, args=(), kwargs={}, group_id=None, chord=None,
                task_id=None, **options):
            if self.app.conf.CELERY_ALWAYS_EAGER:
                return self.apply(args, kwargs, **options)
            options.pop('publisher', None)
            tasks, results = self.prepare_steps(args, kwargs['tasks'])
            result = results[-1]
            if group_id:
                tasks[-1].set(group_id=group_id)
            if chord:
                tasks[-1].set(chord=chord)
            if task_id:
                tasks[-1].set(task_id=task_id)
                result = tasks[-1].type.AsyncResult(task_id)
            tasks[0].apply_async()
            return result

        def apply(self, args=(), kwargs={}, **options):
            tasks = [maybe_subtask(task).clone() for task in kwargs['tasks']]
            res = prev = None
            for task in tasks:
                res = task.apply((prev.get(), ) if prev else ())
                res.parent, prev = prev, res
            return res
    return Chain

您可以看到最后prepare_steps prev_task链接到下一个任务。当 prev_task 失败时，不会调用下一个任务。

我正在测试将 prev 任务中的 link_error 添加到下一个任务：

if prev_task:
    # link and link_error previous task to this task.
    prev_task.link(task)
    prev_task.link_error(task)
    # set the results parent attribute.
    res.parent = prev_res

但是，下一个任务必须处理这两种情况（也许，除非它被配置为不可变，例如不接受更多参数）。

我认为链可以通过允许一些这样的语法来支持这一点：

c = chain(t1, (t2, t1e), (t3, t2e))

意思是：

t1 link去t2和link_error去t1e

t2 link去t3和link_error去t2e

于 2012-07-23T04:37:40.033 回答