DATAREDIS-1013 - Fix wrong argument count for SELECT command

[svolle]

Redis SELECT command requires a single argument.

[pivotal-issuemaster]

@svolle Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

[pivotal-issuemaster]

@svolle Thank you for signing the Contributor License Agreement!

[svolle]

I could not reproduce the test failure in my dev environment and the failed test does not seem related in any way with my change. It looks like a spurious failure to me.

[mp911de]

No worries. Some tests include kind-of invisible latency or timing assumptions that fail when the CI system is slow.

[mp911de]

Can you also provide a test that shows the change is working via RedisCommands.execute(…)?

[svolle]

Sure. Can I assume a redis server running on localhost for this test? It seems to be already the case for other tests.

[mp911de]

Yes, you can. You can start the test environment via make start. Alternatively, $ redis-server (using default config) should be fine. Thanks a lot.

[mp911de]

That's merged now.

[svolle]

Thanks Mark. Should I create another PR when I eventually come around to add the UT you requested?

[mp911de]

I took care of the test during the merge.

[svolle]

My bad, I did not see the new commits. Sorry for being a slowpoke and not taking care of this in a timely manner.

Thanks!