This is an automated email from the ASF dual-hosted git repository.
jiayu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sedona.git
The following commit(s) were added to refs/heads/master by this push:
new 4583e9e26 [SEDONA-618] Fix maven build errors that are ignored in the
parent pom (#1499)
4583e9e26 is described below
commit 4583e9e265157fe2332a7cf52c8348671a099ab4
Author: Feng Zhang <[email protected]>
AuthorDate: Wed Jun 26 10:13:24 2024 -0700
[SEDONA-618] Fix maven build errors that are ignored in the parent pom
(#1499)
* Fix maven build errors that are ignored in the parent pom
* remove fail on error comment
* move Metric to java
* remove config
---
pom.xml | 1 -
.../apache/sedona/core/monitoring/Metrics.java} | 17 ++++++++--------
.../org/apache/sedona/sql/ogc/package-info.java | 23 ----------------------
3 files changed, 9 insertions(+), 32 deletions(-)
diff --git a/pom.xml b/pom.xml
index d627165b9..aa8b2f0f7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -611,7 +611,6 @@
</execution>
</executions>
<configuration>
- <failOnError>false</failOnError>
<additionalparam>-Xdoclint:none</additionalparam>
</configuration>
</plugin>
diff --git
a/spark/common/src/main/scala/org/apache/sedona/core/monitoring/Metrics.scala
b/spark/common/src/main/java/org/apache/sedona/core/monitoring/Metrics.java
similarity index 70%
rename from
spark/common/src/main/scala/org/apache/sedona/core/monitoring/Metrics.scala
rename to
spark/common/src/main/java/org/apache/sedona/core/monitoring/Metrics.java
index 01373c558..c9061436c 100644
---
a/spark/common/src/main/scala/org/apache/sedona/core/monitoring/Metrics.scala
+++ b/spark/common/src/main/java/org/apache/sedona/core/monitoring/Metrics.java
@@ -16,15 +16,16 @@
* specific language governing permissions and limitations
* under the License.
*/
-package org.apache.sedona.core.monitoring
+package org.apache.sedona.core.monitoring;
-import org.apache.spark.SparkContext
-import org.apache.spark.util.LongAccumulator
+import org.apache.spark.SparkContext;
+import org.apache.spark.util.LongAccumulator;
-object Metrics {
- def createMetric(sc: SparkContext, name: String): LongAccumulator = {
- val acc = new LongAccumulator()
- sc.register(acc, "sedona.spatialjoin." + name)
- acc
+public class Metrics {
+
+ public static LongAccumulator createMetric(SparkContext sc, String name) {
+ LongAccumulator acc = new LongAccumulator();
+ sc.register(acc, "sedona.spatialjoin." + name);
+ return acc;
}
}
diff --git
a/spark/common/src/test/scala/org/apache/sedona/sql/ogc/package-info.java
b/spark/common/src/test/scala/org/apache/sedona/sql/ogc/package-info.java
deleted file mode 100644
index ed07598bc..000000000
--- a/spark/common/src/test/scala/org/apache/sedona/sql/ogc/package-info.java
+++ /dev/null
@@ -1,23 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-/**
- * This package contains compliance tests for OGC simple feature access part
two version 1.2.1.
- * https://www.ogc.org/standards/sfs
- */
-package org.apache.sedona.sql.ogc;
