Skip to content

Make sure that a worker picks up a task only if it done with previous. #517

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

Merged
merged 1 commit into from
Sep 19, 2014

Conversation

manasdk
Copy link
Contributor

@manasdk manasdk commented Sep 19, 2014

  • Setting prefetch_count=1 means rabbitmq employs fairdispatch to workers.
    This way unless a worker acknowledges a message a new message will not
    be delivered to the worker. This avoids the case where a blocking task
    or a really long running action does not do a DoS on some unsuspecting
    other action.
  • Modified the launchdev script to include no of workers. By default 1
    actionrunner worker is started.
    Use 'tools/launchdev.sh start N' where N is number of actionrunners.

- Setting prefetch_count=1 means rabbitmq employs fairdispatch to workers.
  This way unless a worker acknowledges a message a new message will not
  be delivered to the worker. This avoids the case where a blocking task
  or a really long running action does not do a DoS on some unsuspecting
  other action.
- Modified the launchdev script to include no of workers. By default 1
  actionrunner worker is started.
  Use 'tools/launchdev.sh start N' where N is number of actionrunners.
@DoriftoShoes
Copy link

LGTM

manasdk pushed a commit that referenced this pull request Sep 19, 2014
Make sure that a worker picks up a task only if it done with previous.
@manasdk manasdk merged commit 9a51b0e into master Sep 19, 2014
@manasdk manasdk deleted the dev/worker-qos branch September 19, 2014 00:15
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

Successfully merging this pull request may close these issues.

2 participants