Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TaskRevokedError Exceptions stored to queue even when CELERY_IGNORE_RESULT=True #207

Closed
rlotun opened this issue Oct 6, 2010 · 1 comment

Comments

@rlotun
Copy link
Contributor

rlotun commented Oct 6, 2010

I'm running Celery 2.0.3 on RabbitMQ 1.7.2

In my celeryconfig.py I have CELERY_IGNORE_RESULT = True however, I see results still pushed to the queue, i.e.:

$ rabbitmqctl list_bindings -p VHOST | grep ^celeryresults | wc -l
59716

and

$ camqadm
1> basic.get fffb6152325f400eb8768dbf94cbdfaa
 {'body': "  (dp1\nS'status'\np2\nS'REVOKED'\np3\nsS'traceback'\np4\nNsS'result'\np5\nccelery.exceptions\nTaskRevokedError\np6\n(tRp7\nsS'task_id'\np8\nVfffb6152-325f-400e-b876-8dbf94cbdfaa\np9\ns.",
 'delivery_info': {'delivery_tag': 1,
               'exchange': u'celeryresults',
               'message_count': 9,
               'redelivered': False,
               'routing_key': u'fffb6152325f400eb8768dbf94cbdfaa'},
 'properties': {'content_encoding': u'binary',
            'content_type': u'application/x-python-serialize',
            'delivery_mode': 1,
            'priority': 0}}

Quite possibly, revoking tasks don't respect the CELERY_IGNORE_RESULT setting.

@ask
Copy link
Contributor

ask commented Oct 7, 2010

Don't mark tasks as revoked if CELERY_IGNORE_RESULT=True. Closed by 36ea925. Thanks to rlotun

heyman pushed a commit to heyman/celery that referenced this issue Sep 3, 2011
heyman pushed a commit to heyman/celery that referenced this issue Sep 3, 2011
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants