seomoz / qless-core

Core Lua Scripts for qless
MIT License
85 stars 34 forks source link

Move job failure messages into separate, hash-based keys #14

Open bkirz opened 11 years ago

bkirz commented 11 years ago

qless-core currently stores a job's failure message as part of its data. For the most worker types, this means all failed jobs have exception backtraces and messages in their job data. We've seen thousands of jobs fail with identical failure messages, which bloats our memory usage considerably. Additionally, these keys never expire, so the only way to prevent running out of memory is to handle these errors manually.

I discussed this with @myronmarston, and we came up with a few changes we can make to solve this problem:

What do you think, @dlecocq?