Do not build keys on lua scripts

[pacoguzman]

Problem

Our one of our projects we're using redis-namespace which below 2.0 try to implicitly namespace keys transparently.

For example for script calls it does the following as an example:

https://github.com/resque/redis-namespace/blob/30d42177d8abfdf89e502c1a98aab7d923c2a133/lib/redis/namespace.rb#L372

Well, we're going to evaluate keeping using redis-namespace on the future, but we cannot afford the migration right now, and we realized that we cannot make use of the new conflict resolution features because of this:

sidekiq-unique-jobs/redis/delete_job_by_digest.lua

Line 1 in 9d1ee6c

local queue = "queue:" .. KEYS[1]

As it's going to build a string just like queue:default when in our redis service should be app_name:queue:default

Solution

We provided a slightly change so the ruby code provides which it thinks is the whole key to search for jobs then redis-namespace will add the namespace transparently.

Please let me know what you think, maybe we should add some kind of not on README about the redis-namespace usage or any other thing to clarify the previous situation and if this is accepted in some way evaluate if it's correct or not.

[mhenrixon]

At first glance it looks good. I'll have to dig into this a little deeper before but I have no objections against this.