Skip to content

Fix for Issue #6 & #2#7

Open
torotil wants to merge 3 commits into
RentMethod:masterfrom
torotil:issue6
Open

Fix for Issue #6 & #2#7
torotil wants to merge 3 commits into
RentMethod:masterfrom
torotil:issue6

Conversation

@torotil

@torotil torotil commented Apr 8, 2015

Copy link
Copy Markdown

No description provided.

@torotil

torotil commented May 18, 2015

Copy link
Copy Markdown
Author

I'm unsure what the behavior of active and idle should be wrt to scheduled tasks. This pull request changes the behavior because it sets the worker to active as soon as at least one task is in the queue (scheduled or not). As the bug report (#6) shows the previous behavior is broken too because there is no way to wait for all tasks to have finished. I think the later is the far more important use-case.

@halfak

halfak commented Jun 30, 2016

Copy link
Copy Markdown

I would love to get this PR merged as python 3 compatibility is blocker for me.

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