fix tests

Author: zhangfengcdt

parquet-column/src/main/java/org/apache/parquet/column/statistics/geometry/GeospatialStatistics.java

@@ -62,8 +62,7 @@ public void update(Binary value) {
         Geometry geom = reader.read(value.getBytes());
         update(geom);
       } catch (ParseException e) {
-        LOG.warn("Failed to parse WKB geometry, aborting statistics update", e);
-        abort();
+        LOG.warn("Failed to parse WKB geometry, omit it from stats", e);
       }
     }
 

parquet-column/src/test/java/org/apache/parquet/column/statistics/geometry/TestGeospatialStatistics.java

@@ -98,8 +98,8 @@ public void testMergeNullGeospatialStatistics() {
     GeospatialStatistics validCopy = validStats.copy();
     validCopy.merge(nullStats);
     Assert.assertFalse(validCopy.isValid());
-    Assert.assertNull(validCopy.getBoundingBox());
-    Assert.assertNull(validCopy.getGeospatialTypes());
+    Assert.assertNotNull(validCopy.getBoundingBox());
+    Assert.assertNotNull(validCopy.getGeospatialTypes());
 
     // Test merging null with valid
     nullStats = new GeospatialStatistics(null, null);
@@ -115,8 +115,8 @@ public void testMergeNullGeospatialStatistics() {
     // Test merging valid with null bounding box
     validCopy = validStats.copy();
     validCopy.merge(nullBboxStats);
-    Assert.assertFalse(validCopy.isValid());
-    Assert.assertNull(validCopy.getBoundingBox());
+    Assert.assertTrue(validCopy.isValid());
+    Assert.assertNotNull(validCopy.getBoundingBox());
     Assert.assertNotNull(validCopy.getGeospatialTypes());
   }
 

parquet-hadoop/src/test/java/org/apache/parquet/statistics/TestGeometryTypeRoundTrip.java

@@ -165,7 +165,7 @@ public void testBasicReadWriteGeographyValue() throws Exception {
       Assert.assertNotNull(columnIndex);
 
       GeospatialStatistics geospatialStatistics = columnChunkMetaData.getGeospatialStatistics();
-      Assert.assertNotNull(geospatialStatistics);
+      Assert.assertNull(geospatialStatistics);
     }
   }