ERR EXEC without MULTI - Jedis

Question

I'm learning jedis, I couldn't exactly find out what is the problem in this code. Can anyone help me.The exception occurs at the statement tx.exec()

public class JedisFactory {

 public static void main (String [] args){
     JedisPool pool = new JedisPool(new JedisPoolConfig(), "127.0.0.1", 6379);
     Jedis jedis = pool.getResource();

     Pipeline pipeline = jedis.pipelined();
     for(int i=0; i < 1000 ; i++){
         pipeline.hincrBy("Id", i+"", i);
     }
     pipeline.exec();        
     pool.returnResource(jedis);

     jedis = pool.getResource();
     Transaction tx = jedis.multi();
     Response<Map<String,String>> map = tx.hgetAll("Id");
     tx.hincrBy("Id","2", 1);
     **tx.exec();**
     //Map<String,String> map1 = jedis.hgetAll("Id");

     pool.returnResource(jedis);
     pool.destroy();
 }
}
Exception in thread "main" redis.clients.jedis.exceptions.JedisDataException: ERR EXEC without MULTI
    at redis.clients.jedis.Protocol.processError(Protocol.java:54)
    at redis.clients.jedis.Protocol.process(Protocol.java:61)
    at redis.clients.jedis.Protocol.read(Protocol.java:122)
    at redis.clients.jedis.Connection.getAll(Connection.java:207)
    at redis.clients.jedis.BinaryTransaction.exec(BinaryTransaction.java:23)
    at com.work.JedisFactory.main(JedisFactory.java:30)

Answer 1

Adding pipeline.multi() before for loop solved the problem. But the exception thrown at some other line before the fix.

Answer 2

You should surround pipeline.exec(); with multi() and close() methods. Like this:

pipeline.multi() ;
pipeline.exec();
pipeline.close(); 

Answer 3

I guess you should use pipeline.execute() instead of pipeline.exec(), at least I used it like that and it was ok.