Merge pull request #35 from aliyun/feature/0.2.4

Fixed LastPointQuery CircularReference bug

Author: Rocky0714

pom.xml

@@ -3,7 +3,7 @@
 	<modelVersion>4.0.0</modelVersion>
 	<groupId>com.aliyun</groupId>
 	<artifactId>hitsdb-client</artifactId>
-	<version>0.2.3</version>
+	<version>0.2.4</version>
 	<name>Aliyun TSDB SDK for Java</name>
 	<description>The Alibaba TSDB SDK for Java used for accessing High Performance Time Series Database</description>
 	<url>https://www.aliyun.com/product/hitsdb</url>

src/main/java/com/aliyun/hitsdb/client/value/JSONValue.java

@@ -1,6 +1,7 @@
 package com.aliyun.hitsdb.client.value;
 
 import com.alibaba.fastjson.JSON;
+import com.alibaba.fastjson.serializer.SerializerFeature;
 
 public class JSONValue {
 
@@ -17,7 +18,7 @@ public static <T> List<T> parseList(String json, Class<T> clazz) {
     */
 
     public String toJSON() {
-        return JSON.toJSONString(this);
+        return JSON.toJSONString(this, SerializerFeature.DisableCircularReferenceDetect);
     }
 
     @Override

src/test/java/com/aliyun/hitsdb/client/LastPointQueryTest.java

@@ -0,0 +1,27 @@
+package com.aliyun.hitsdb.client;
+
+import com.aliyun.hitsdb.client.value.request.LastPointQuery;
+import com.aliyun.hitsdb.client.value.request.LastPointSubQuery;
+import org.junit.Test;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.junit.Assert.assertEquals;
+
+public class LastPointQueryTest {
+
+    @Test
+    public void testNotCircularReference() {
+        Map<String, String> tags = new HashMap<String, String>();
+        tags.put("k1", "v1");
+
+        LastPointQuery lastPointQuery = LastPointQuery
+                .builder()
+                .sub(LastPointSubQuery.builder("m1", tags).build())
+                .sub(LastPointSubQuery.builder("m2", tags).build()).build();
+
+        String expected = "{\"queries\":[{\"metric\":\"m1\",\"tags\":{\"k1\":\"v1\"}},{\"metric\":\"m2\",\"tags\":{\"k1\":\"v1\"}}]}";
+        assertEquals(expected, lastPointQuery.toString());
+    }
+}