TWEAK: use rpoplpush in lua script

Because it's more clear and slightly faster. Increased jobs/sec by about 600.

README.md

@@ -257,7 +257,7 @@ The benches folder contains various benchmark code. In each case, we enqueue 100
 
 | Library | Speed |
 | --- | --- |
-| [gocraft/work](https://www.github.com/gocraft/work) | **20351 jobs/s** |
+| [gocraft/work](https://www.github.com/gocraft/work) | **20944 jobs/s** |
 | [jrallison/go-workers](https://www.github.com/jrallison/go-workers) | 19945 jobs/s |
 | [benmanns/goworker](https://www.github.com/benmanns/goworker) | 10328.5 jobs/s |
 | [albrow/jobs](https://www.github.com/albrow/jobs) | 40 jobs/s |

redis.go

@@ -56,9 +56,8 @@ var redisLuaRpoplpushMultiCmd = `
 local res
 local keylen = #KEYS
 for i=1,keylen,2 do
-  res = redis.call('rpop', KEYS[i])
+  res = redis.call('rpoplpush', KEYS[i], KEYS[i+1])
   if res then
-    redis.call('lpush', KEYS[i+1], res)
     return {res, KEYS[i], KEYS[i+1]}
   end
 end

todo.txt

@@ -14,13 +14,13 @@ go run main.go
 
 TODO
 ----
+ - awesomego PR
  - readme:
    - webui
-   - references
- - Benchmark latest version of goworker and go-workers again and that other one
  - once I have thruput benches in place:
    - rpoplpush in lua script
    - zero context each time
+ - do a benchmark to measure memory allocations
 
  - benchmarks for runJob
  - investigate changing runJob to use a shared context, and zero'ing it's value each time