Addressed PR comments

Signed-off-by: Guian Gumpac <[email protected]>
opensearch-project · Jul 27, 2023 · d483491 · d483491
1 parent 61aaa8f
commit d483491
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/opensearch/src/main/java/org/opensearch/sql/opensearch/data/type/OpenSearchGeoPointType.java b/opensearch/src/main/java/org/opensearch/sql/opensearch/data/type/OpenSearchGeoPointType.java
@@ -31,10 +31,6 @@ private OpenSearchGeoPointType() {
     this.properties.put("lon", new OpenSearchDataType(ExprCoreType.DOUBLE));
   }
 
-  @Getter
-  @EqualsAndHashCode.Exclude
-  Map<String, OpenSearchDataType> properties;
-
   public static OpenSearchGeoPointType of() {
     return OpenSearchGeoPointType.instance;
   }

diff --git a/opensearch/src/main/java/org/opensearch/sql/opensearch/data/utils/OpenSearchJsonContent.java b/opensearch/src/main/java/org/opensearch/sql/opensearch/data/utils/OpenSearchJsonContent.java
@@ -149,6 +149,7 @@ public Pair<Double, Double> geoValue() {
       }
     }
 
+    // lat or lon can be null to support geopoint.lat query but both can't be null
     if (lat == null && lon == null) {
       throw new IllegalStateException("geo point must be in format of {\"lat\": number, \"lon\": "
           + "number}");