Closed nidico closed 10 years ago
Looks fine to me. I can also not see any reason why the queue should not be cached. However, I do not feel like I understand enough about this to merge it.
This isn't urgent and I don't know whether it helps much anyway, so we don't need to merge this immediately. If someone's up to looking into this more deeply, feel welcome.
This makes sure the
rq.Queue
object is only created once.I haven't checked on performance gains, but it feels wrong reconstructing the queue over and over again. In my understanding caching the queue should be save.