Redis.select() requires host and port

[mjrusso]

From the documentation for the select method of Redis:

Switch to a different database on the current host/port

However, host and port are required parameters:

def select(self, host, port, db, password=None, socket_timeout=None):

Can we make host and port optional, with db the only required parameter?

[andymccurdy]

The additional arguments were intentionally added to make the user realize that they're switching the underlying TCP connection whenever they switch the database.

I suppose the current host/port could be used if changing to a different database and those arguments weren't provided.

[mjrusso]

I suppose the current host/port could be used if changing to a different database and those arguments weren't provided.

That's what I would have expected. However, as a user, it isn't a big deal to imitate that behavior with the existing implementation.

In any event, I would recommend either changing the documentation or the method signature/ semantics so that the two are in sync.

[andymccurdy]

Ok, this has been fixed and commited in c148ede