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

Already on GitHub? Sign in to your account

Parallelize stop and restart to be faster with many workers #384

Merged
merged 1 commit into from May 12, 2013

Conversation

Projects
None yet
3 participants

Stoping delayed job via deamons gem is taking ages with many workers, because it sends signal sequentially to each process. In my case, I have 15 workers and have to wait 15 * 20 seconds (timeout duration) to restart all workers.

My solution is to send each signal in their own thread. Thus, restart duration doesn't increase as many as worker count.

Looks good, any reason why this commit has not been merged yet?

Collaborator

sferik commented May 12, 2013

Looks good to me.

@sferik sferik added a commit that referenced this pull request May 12, 2013

@sferik sferik Merge pull request #384 from alexisbernard/parallelize_restart
Parallelize stop and restart to be faster with many workers
4aa0bc5

@sferik sferik merged commit 4aa0bc5 into collectiveidea:master May 12, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment