delete() take in a list of keys

The use case came up when I tried to do this:

# conn is a valid StrictRedis instance
keys = conn.keys('namespace:*')  # this returns a list
conn.delete(keys)  # does not delete all the keys in the list

conn.delete(*keys)  # raises ResponseError: wrong number of arguments for 'del' command

# edit: conn.delete(*keys) works when the list is not empty

Can we make it a bit neater to allow delete to just accept a list or a set of strings (keys) to do the necessary check to be passed to execute_command('DEL', ...)?

[brejoc]

I ran into this issue last week in the context of the redis returner in SaltStack. It took me some time to dig out the true source for this error, because I was not aware, that Salt tried to pass an empty list. A slightly modified error message would have been a huge benefit.

[woozyking]

It is pythonic, but when you do conn.delete(*list_of_keys), list_of_keys must not be empty or it’ll raise ResponseError: wrong number of arguments for 'del' command.

I’m just proposing for a slightly more convenient interface for users who can just pass in a list or a set of strings to be deleted.