Skip to content

New thread-pool-based worker - #123

Merged
nesquena merged 6 commits into
nesquena:masterfrom
contentfree:master
Apr 22, 2016
Merged

New thread-pool-based worker#123
nesquena merged 6 commits into
nesquena:masterfrom
contentfree:master

Conversation

@contentfree

Copy link
Copy Markdown
Collaborator

Here's the new worker, @nesquena.

@nesquena

Copy link
Copy Markdown
Owner

This is great, thanks Dave! Any ideas as to why travis always fails during the install now? Would be nice to get the tests passing again :(

@nesquena
nesquena merged commit 5b3b06a into nesquena:master Apr 22, 2016
@contentfree

contentfree commented Apr 22, 2016 via email

Copy link
Copy Markdown
Collaborator Author

@nesquena

Copy link
Copy Markdown
Owner

Can you confirm tests are passing locally again now?

@geoffroymontel

Copy link
Copy Markdown

Hello !
I submitted also a thread based worker recently, is it comparable or should I drop it ? :)
#120
All the best

@nesquena

Copy link
Copy Markdown
Owner

@contentfree how does his threaded worker compare to the one we just merged?

@contentfree

Copy link
Copy Markdown
Collaborator Author

It's similar, though the thread pool will likely be more resilient. @geoffroymontel's spins up the threads once (as my original fixed thread pool implementation did) and it won't self-heal like the latest thread pool implementation will.

I do like his Thread.current use will see if I can put it to use in a similar way.

Re: the tests – Strangely, running the full test suite fails (several failures in test/workers/simple_worker_test.rb, for example) but they don't fail when run individually (at least not simple_worker_test.rb). Looks like we're not resetting things quite right. Haven't found the cause yet, though…

@geoffroymontel

Copy link
Copy Markdown

Indeed my code was not resilient. I was not aware of the ruby-concurrency gem, thanks for letting me know about it !

ramkumar-kr added a commit to ramkumar-kr/backburner that referenced this pull request Sep 11, 2016
This reverts commit 5b3b06a, reversing
changes made to d161619.
ramkumar-kr added a commit to ramkumar-kr/backburner that referenced this pull request Sep 24, 2016
Revert "Merge pull request nesquena#123 from contentfree/master"
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.

3 participants