Geo: Add GeoJson parser to libs/geo classes

Adds GeoJson parser for Geometry classes defined in libs/geo. This
should allow us to bypass ShapeBuilders in geosql and is the first
step in ShapeBuilder refactoring. The provided functionality should
be sufficient for geosql parsing, but additional functionality will
be required for complete parity with ShapeBuilder parser - we need
to add handling of mapper parameters in WKT parser as well. That
will be added in a follow up PR.

Relates elastic#40908

Author: imotov

libs/geo/src/main/java/org/elasticsearch/geo/geometry/Circle.java

@@ -102,7 +102,7 @@ public int hashCode() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 

libs/geo/src/main/java/org/elasticsearch/geo/geometry/Geometry.java

@@ -26,7 +26,7 @@ public interface Geometry {
 
     ShapeType type();
 
-    <T> T visit(GeometryVisitor<T> visitor);
+    <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E;
 
     boolean isEmpty();
 

libs/geo/src/main/java/org/elasticsearch/geo/geometry/GeometryCollection.java

@@ -57,7 +57,7 @@ public ShapeType type() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 

libs/geo/src/main/java/org/elasticsearch/geo/geometry/GeometryVisitor.java

@@ -44,26 +44,26 @@
  *
  * @see <a href="https://en.wikipedia.org/wiki/Visitor_pattern">Visitor Pattern</a>
  */
-public interface GeometryVisitor<T> {
+public interface GeometryVisitor<T, E extends Exception> {
 
-    T visit(Circle circle);
+    T visit(Circle circle) throws E;
 
-    T visit(GeometryCollection<?> collection);
+    T visit(GeometryCollection<?> collection) throws E;
 
-    T visit(Line line);
+    T visit(Line line) throws E;
 
-    T visit(LinearRing ring);
+    T visit(LinearRing ring) throws E;
 
-    T visit(MultiLine multiLine);
+    T visit(MultiLine multiLine) throws E;
 
-    T visit(MultiPoint multiPoint);
+    T visit(MultiPoint multiPoint) throws E;
 
-    T visit(MultiPolygon multiPolygon);
+    T visit(MultiPolygon multiPolygon) throws E;
 
-    T visit(Point point);
+    T visit(Point point) throws E;
 
-    T visit(Polygon polygon);
+    T visit(Polygon polygon) throws E;
 
-    T visit(Rectangle rectangle);
+    T visit(Rectangle rectangle) throws E;
 
 }

libs/geo/src/main/java/org/elasticsearch/geo/geometry/Line.java

@@ -103,7 +103,7 @@ public ShapeType type() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 

libs/geo/src/main/java/org/elasticsearch/geo/geometry/LinearRing.java

@@ -54,7 +54,7 @@ public ShapeType type() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 }

libs/geo/src/main/java/org/elasticsearch/geo/geometry/MultiLine.java

@@ -40,7 +40,7 @@ public ShapeType type() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 }

libs/geo/src/main/java/org/elasticsearch/geo/geometry/MultiPoint.java

@@ -40,7 +40,7 @@ public ShapeType type() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 

libs/geo/src/main/java/org/elasticsearch/geo/geometry/MultiPolygon.java

@@ -40,7 +40,7 @@ public ShapeType type() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 }

libs/geo/src/main/java/org/elasticsearch/geo/geometry/Point.java

@@ -93,7 +93,7 @@ public int hashCode() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 

libs/geo/src/main/java/org/elasticsearch/geo/geometry/Polygon.java

@@ -92,7 +92,7 @@ public LinearRing getHole(int i) {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 

libs/geo/src/main/java/org/elasticsearch/geo/geometry/Rectangle.java

@@ -207,7 +207,7 @@ public int hashCode() {
     }
 
     @Override
-    public <T> T visit(GeometryVisitor<T> visitor) {
+    public <T, E extends Exception> T visit(GeometryVisitor<T, E> visitor) throws E {
         return visitor.visit(this);
     }
 

libs/geo/src/main/java/org/elasticsearch/geo/geometry/ShapeType.java

@@ -19,6 +19,8 @@
 
 package org.elasticsearch.geo.geometry;
 
+import java.util.Locale;
+
 /**
  * Shape types supported by elasticsearch
  */
@@ -33,4 +35,8 @@ public enum ShapeType {
     LINEARRING, // not serialized by itself in WKT or WKB
     ENVELOPE, // not part of the actual WKB spec
     CIRCLE; // not part of the actual WKB spec
+
+    public static ShapeType forName(String geoshapename) {
+        return ShapeType.valueOf(geoshapename.toUpperCase(Locale.ROOT));
+    }
 }

libs/geo/src/main/java/org/elasticsearch/geo/utils/WellKnownText.java

@@ -68,7 +68,7 @@ public static void toWKT(Geometry geometry, StringBuilder sb) {
         if (geometry.isEmpty()) {
             sb.append(EMPTY);
         } else {
-            geometry.visit(new GeometryVisitor<Void>() {
+            geometry.visit(new GeometryVisitor<Void, RuntimeException>() {
                 @Override
                 public Void visit(Circle circle) {
                     sb.append(LPAREN);
@@ -543,7 +543,7 @@ private static String nextCloserOrComma(StreamTokenizer stream) throws IOExcepti
     }
 
     public static String getWKTName(Geometry geometry) {
-        return geometry.visit(new GeometryVisitor<String>() {
+        return geometry.visit(new GeometryVisitor<String, RuntimeException>() {
             @Override
             public String visit(Circle circle) {
                 return "circle";

libs/geo/src/test/java/org/elasticsearch/geo/geometry/BaseGeometryTestCase.java

@@ -67,7 +67,7 @@ public void testVisitor() {
 
     public static void testVisitor(Geometry geom) {
         AtomicBoolean called = new AtomicBoolean(false);
-        Object result = geom.visit(new GeometryVisitor<Object>() {
+        Object result = geom.visit(new GeometryVisitor<Object, RuntimeException>() {
             private Object verify(Geometry geometry, String expectedClass) {
                 assertFalse("Visitor should be called only once", called.getAndSet(true));
                 assertSame(geom, geometry);