TairGis: fix gissearch result factory npe

tair-opensource · Nov 25, 2022 · f621593 · f621593
1 parent 3e85818
commit f621593
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/main/java/com/aliyun/tair/tairgis/factory/GisBuilderFactory.java b/src/main/java/com/aliyun/tair/tairgis/factory/GisBuilderFactory.java
@@ -19,7 +19,7 @@ public Map<String, String> build(Object data) {
             List<byte[]> rawResults;
             List<Object> result = (List<Object>) data;
             if (null == result || 0 == result.size()) {
-                rawResults = null;
+                return new HashMap<>();
             } else {
                 rawResults = (List) result.get(1);
             }
@@ -45,7 +45,7 @@ public Map<byte[], byte[]> build(Object data) {
             List<byte[]> rawResults;
             List<Object> result = (List<Object>) data;
             if (null == result || 0 == result.size()) {
-                rawResults = null;
+                return new HashMap<>();
             } else {
                 rawResults = (List) result.get(1);
             }

diff --git a/src/test/java/com/aliyun/tair/tests/tairgis/TairGisPipelineTest.java b/src/test/java/com/aliyun/tair/tests/tairgis/TairGisPipelineTest.java
@@ -75,6 +75,16 @@ public void gissearchTest() throws Exception {
         AssertUtil.assertEquals(1, Map.class.cast(objs.get(i)).size());
     }
 
+    @Test
+    public void gissearchNotExistsTest() throws Exception {
+        String polygonWktText = "POLYGON ((30 10, 40 40, 20 40, 10 20, 30 10))", pointWktText = "POINT (30 11)";
+        // String
+        tairGisPipeline.gissearch(area, pointWktText);
+        List<Object> objs = tairGisPipeline.syncAndReturnAll();
+        Assert.assertEquals(1, objs.size());
+        Assert.assertEquals(0, Map.class.cast(objs.get(0)).size());
+    }
+
     @Test
     public void gisdelTest() throws Exception {
         String uuid = UUID.randomUUID().toString();