add nextQueueStrategy feature to WorkerPoolImpl

gresrun / jesque

An implementation of Resque in Java.

http://gresrun.github.io/jesque

Apache License 2.0

630 stars 131 forks source link

add nextQueueStrategy feature to WorkerPoolImpl #132

Closed bp-FLN closed 6 years ago

bp-FLN commented 6 years ago

The nextQueueStrategy feature was added in #107, but only for WorkerImpl. This patch adds the feature also to the WorkerPoolImpl.

bp-FLN commented 6 years ago

What about having some common base class for the WorkerImpl and WorkerPoolImpl? Like AbstractWorker or smth similar. Would save us quite some duplicated code. What do you think @gresrun?

gresrun commented 6 years ago

Florian, I agree, the two should share a common base class. Should have done that work when I first created WorkerPoolImpl but I guess I got lazy!