search

Grails-Plugin-Consortium / grails-jesque

1 stars 5 forks source link

fixes #9 #10

Closed peh closed 7 years ago

peh commented 7 years ago

this should fix #9

I groovyfied the WorkerImpl as i had to touch a lot of code anyways. As soon as https://github.com/gresrun/jesque/issues/109 we should consider getting rid of that "custom" WorkerImpl completly and use the original WorkerPoolImpl.

I also cleaned up some stuff that was never used in the custom WorkerImpl

ctoestreich commented 7 years ago

I was going to do this today as we had to switch to a Jedis scan vs key and I realized that the connection needs an explicit close. Will review asap.

peh commented 7 years ago

sorry wrong button :)

ctoestreich commented 7 years ago

We really need more tests around this thing.... releasing now 1.1.9.

ctoestreich commented 7 years ago

thanks