From 443686b15533d3e5d9cd8041ab19d409978f30fc Mon Sep 17 00:00:00 2001
From: Jonathan Leibiusky <jonathan.leibiusky@mercadolibre.com>
Date: Sat, 16 Apr 2011 17:06:00 -0300
Subject: [PATCH] double builder should build from string

---
 .../redis/clients/jedis/BuilderFactory.java   |  2 +-
 .../jedis/tests/BuilderFactoryTest.java       | 14 +++++++++++++
 .../clients/jedis/tests/PipeliningTest.java   | 21 +++++++++++++++++++
 3 files changed, 36 insertions(+), 1 deletion(-)
 create mode 100644 src/test/java/redis/clients/jedis/tests/BuilderFactoryTest.java

diff --git a/src/main/java/redis/clients/jedis/BuilderFactory.java b/src/main/java/redis/clients/jedis/BuilderFactory.java
index 0dca5b6..fcbcd5c 100644
--- a/src/main/java/redis/clients/jedis/BuilderFactory.java
+++ b/src/main/java/redis/clients/jedis/BuilderFactory.java
@@ -14,7 +14,7 @@ import redis.clients.util.SafeEncoder;
 public class BuilderFactory {
     public static final Builder<Double> DOUBLE = new Builder<Double>() {
         public Double build(Object data) {
-            return Double.valueOf((Long) data);
+            return Double.valueOf(STRING.build(data));
         }
 
         public String toString() {
diff --git a/src/test/java/redis/clients/jedis/tests/BuilderFactoryTest.java b/src/test/java/redis/clients/jedis/tests/BuilderFactoryTest.java
new file mode 100644
index 0000000..b759134
--- /dev/null
+++ b/src/test/java/redis/clients/jedis/tests/BuilderFactoryTest.java
@@ -0,0 +1,14 @@
+package redis.clients.jedis.tests;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+import redis.clients.jedis.BuilderFactory;
+
+public class BuilderFactoryTest extends Assert {
+    @Test
+    public void buildDouble() {
+        Double build = BuilderFactory.DOUBLE.build("1.0".getBytes());
+        assertEquals(new Double(1.0), build);
+    }
+}
\ No newline at end of file
diff --git a/src/test/java/redis/clients/jedis/tests/PipeliningTest.java b/src/test/java/redis/clients/jedis/tests/PipeliningTest.java
index ea1c8e3..3c6eaca 100644
--- a/src/test/java/redis/clients/jedis/tests/PipeliningTest.java
+++ b/src/test/java/redis/clients/jedis/tests/PipeliningTest.java
@@ -1,6 +1,7 @@
 package redis.clients.jedis.tests;
 
 import java.io.UnsupportedEncodingException;
+import java.util.HashMap;
 import java.util.List;
 import java.util.Set;
 
@@ -84,4 +85,24 @@ public class PipeliningTest extends Assert {
         string.get();
         p.sync();
     }
+
+    @Test
+    public void lala() {
+
+        Pipeline p = jedis.pipelined();
+
+        HashMap<String, Integer> mydata = new HashMap<String, Integer>();
+
+        mydata.put("test1", 1);
+        mydata.put("test2", 1);
+        mydata.put("test3", 1);
+        mydata.put("test4", 1);
+        mydata.put("test5", 1);
+
+        for (String txtfield : mydata.keySet()) {
+            p.zadd("somekey", mydata.get(txtfield), txtfield);
+            p.zincrby("SUPERUNION", mydata.get(txtfield), txtfield);
+        }
+        p.sync();
+    }
 }