RedisClient not closed when RedisCommands forgot to quit.

[kentyeh]

When following code was running, redisClient meet a EOFException and program hang on.

RedisServer redis = RedisServer.newRedisServer(6379);
redis.start();
try {
    RedisClient redisClient = RedisClient.create("redis://passwd@127.0.0.1:6379/0");
    try (StatefulRedisConnection<String, String> connection = redisClient.connect()) {
        RedisCommands<String, String> syncCommands = connection.sync();
        syncCommands.set("key", "Hello World!");
        System.out.printf("%s\n", syncCommands.get("key"));
        //syncCommands.quit();
    }
    redisClient.shutdown();
} finally {
    redis.stop();
}

Please add socket close check to RedisClient's run() procedure
from

while (running.get()) {
    Optional<RedisCommand> command = nextCommand();
    ...

to

while (running.get()) {
    if (socket.isClosed()){
        close(); //resources release here.
    }else{
        Optional<RedisCommand> command = nextCommand();
    ...

[inponomarev]

Hi @kentyeh , thanks for your proposal, expect the fix in the next release