xetorthio/jedis — pull request #770 by rdifalco (2014-10-18 19:32:22+00:00)
Title: Fixed infinite timeout handling. Increased code reuse so that there will...

... be fewer copy and paste errors.

This primarily fixes issues where setTimeoutInfinite was called without rollbackTimeout OR rollbackTimeout not being called in a finally block. It was also changing the setTcpNoDelay which was already being set when the socket was constructed.

I also fixed some broken windows on Javadocs.

There still seems to be some missing usages of checkInMulti throughout the code -- but I tried to stay on task. This code could REALLY benefit from refactoring Jedis to delegate to BinaryJedis and then keep all the connection handling et all in BinaryJedis. Currently there is a LOT of copy and paste code. I will try to address this in a future contribution.