(sedona) branch master updated: [GH-2465] Log4j Class Initialization Deadlock in SedonaKryoRegistrator Class. Closes issue #2465 (#2467)

[jiayu]

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 842dfe7aed [GH-2465] Log4j Class Initialization Deadlock in 
SedonaKryoRegistrator Class. Closes issue #2465 (#2467)
842dfe7aed is described below

commit 842dfe7aedf20ea2b72ca4f103074f7e1c90755e
Author: Joonas Pessi <joonas.pe...@mapbox.com>
AuthorDate: Sat Nov 1 09:08:43 2025 +0200

    [GH-2465] Log4j Class Initialization Deadlock in SedonaKryoRegistrator 
Class. Closes issue #2465 (#2467)
---
 .../java/org/apache/sedona/core/serde/SedonaKryoRegistrator.java     | 5 -----
 .../org/apache/sedona/viz/core/Serde/SedonaVizKryoRegistrator.java   | 4 +---
 2 files changed, 1 insertion(+), 8 deletions(-)

diff --git 
a/spark/common/src/main/java/org/apache/sedona/core/serde/SedonaKryoRegistrator.java
 
b/spark/common/src/main/java/org/apache/sedona/core/serde/SedonaKryoRegistrator.java
index 7ea81173f8..04937ea9bf 100644
--- 
a/spark/common/src/main/java/org/apache/sedona/core/serde/SedonaKryoRegistrator.java
+++ 
b/spark/common/src/main/java/org/apache/sedona/core/serde/SedonaKryoRegistrator.java
@@ -19,7 +19,6 @@
 package org.apache.sedona.core.serde;
 
 import com.esotericsoftware.kryo.Kryo;
-import org.apache.log4j.Logger;
 import org.apache.sedona.common.geometryObjects.Circle;
 import org.apache.sedona.common.geometrySerde.GeometrySerde;
 import org.apache.sedona.common.geometrySerde.SpatialIndexSerde;
@@ -37,15 +36,11 @@ import org.locationtech.jts.index.strtree.STRtree;
 
 public class SedonaKryoRegistrator implements KryoRegistrator {
 
-  static final Logger log = Logger.getLogger(SedonaKryoRegistrator.class);
-
   @Override
   public void registerClasses(Kryo kryo) {
     GeometrySerde serializer = new GeometrySerde();
     SpatialIndexSerde indexSerializer = new SpatialIndexSerde(serializer);
 
-    log.info("Registering custom serializers for geometry types");
-
     kryo.register(Point.class, serializer);
     kryo.register(LineString.class, serializer);
     kryo.register(Polygon.class, serializer);
diff --git 
a/spark/common/src/main/java/org/apache/sedona/viz/core/Serde/SedonaVizKryoRegistrator.java
 
b/spark/common/src/main/java/org/apache/sedona/viz/core/Serde/SedonaVizKryoRegistrator.java
index 1e0e4b22b6..e6547f92d2 100644
--- 
a/spark/common/src/main/java/org/apache/sedona/viz/core/Serde/SedonaVizKryoRegistrator.java
+++ 
b/spark/common/src/main/java/org/apache/sedona/viz/core/Serde/SedonaVizKryoRegistrator.java
@@ -19,14 +19,12 @@
 package org.apache.sedona.viz.core.Serde;
 
 import com.esotericsoftware.kryo.Kryo;
-import org.apache.log4j.Logger;
 import org.apache.sedona.core.serde.SedonaKryoRegistrator;
 import org.apache.sedona.viz.core.ImageSerializableWrapper;
 import org.apache.sedona.viz.utils.Pixel;
 import org.apache.spark.serializer.KryoRegistrator;
 
 public class SedonaVizKryoRegistrator implements KryoRegistrator {
-  static final Logger log = Logger.getLogger(SedonaVizKryoRegistrator.class);
 
   @Override
   public void registerClasses(Kryo kryo) {
@@ -34,7 +32,7 @@ public class SedonaVizKryoRegistrator implements 
KryoRegistrator {
     ImageWrapperSerializer imageWrapperSerializer = new 
ImageWrapperSerializer();
     PixelSerializer pixelSerializer = new PixelSerializer();
     sedonaKryoRegistrator.registerClasses(kryo);
-    log.info("Registering custom serializers for visualization related types");
+
     kryo.register(ImageSerializableWrapper.class, imageWrapperSerializer);
     kryo.register(Pixel.class, pixelSerializer);
   }