search

Grails-Plugin-Consortium / grails-jesque

1 stars 5 forks source link

use Pool* classes provided by jesque #28

Closed bp-FLN closed 6 years ago

bp-FLN commented 6 years ago

Jesque already provides classes that can handle a redisPool instead of using it's own connection. Seems like the code was implemented before jesque had those classes, so it's not needed anymore.

Also retrofitted the GrailsJesqueJobFactory so there is no reason to overwrite the workers process method anymore.

peh commented 6 years ago

Yeah when the pooling was implemented the AdminClientPoolImpl was missing from jesque^^

bp-FLN commented 6 years ago

any chance on having this merged? we got some more patches which we would like to incorporate into the upstream repository

thanks