command_builder should should allow SADD to pass an array as an argument

breamware / sidekiq-batch

Sidekiq Batch Jobs Implementation

MIT License

357 stars 95 forks source link

command_builder should should allow SADD to pass an array as an argument #65

Closed jpaas closed 1 year ago

jpaas commented 1 year ago

Using 0.1.9 when I add my job to the batch I get this error: Unsupported command argument type: Array because it passes the job ids to the SADD command as an array but the argument validation in command_builder does not allow this.

Fix is trivial. PR to follow.

jpaas commented 1 year ago

Oops sorry, wrong library. The command_builder file in question comes from redis-client. My bad. I recently upgraded to sidekiq 7 which now uses redis-client and encountered this issue. I will log it with them. Sorry.