(hive) branch master updated: HIVE-28304: Replace org.apache.log4j.Logger with org.slf4j.Logger (#5280)(seonggon, reviewed by Butao Zhang)

[zhangbutao]

This is an automated email from the ASF dual-hosted git repository.

zhangbutao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new 7f87a3b0ef5 HIVE-28304: Replace org.apache.log4j.Logger with 
org.slf4j.Logger (#5280)(seonggon, reviewed by Butao Zhang)
7f87a3b0ef5 is described below

commit 7f87a3b0ef5bd468df34fb4dd5bb4c4db2ac2245
Author: seonggon <ln...@postech.ac.kr>
AuthorDate: Tue Jun 18 12:17:30 2024 +0900

    HIVE-28304: Replace org.apache.log4j.Logger with org.slf4j.Logger 
(#5280)(seonggon, reviewed by Butao Zhang)
---
 .../org/apache/hadoop/hive/accumulo/TestHiveAccumuloHelper.java    | 5 +++--
 .../org/apache/hadoop/hive/ql/udf/esri/HiveGeometryOIHelper.java   | 7 ++++---
 .../org/apache/hadoop/hive/ql/udf/esri/ST_GeometryRelational.java  | 5 +++--
 3 files changed, 10 insertions(+), 7 deletions(-)

diff --git 
a/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/TestHiveAccumuloHelper.java
 
b/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/TestHiveAccumuloHelper.java
index af561e0c3d4..e9ac2e1d1bd 100644
--- 
a/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/TestHiveAccumuloHelper.java
+++ 
b/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/TestHiveAccumuloHelper.java
@@ -31,14 +31,15 @@ import org.apache.hadoop.mapred.JobConf;
 import org.apache.hadoop.security.UserGroupInformation;
 import org.apache.hadoop.security.token.Token;
 import org.apache.hadoop.security.token.TokenIdentifier;
-import org.apache.log4j.Logger;
 import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mockito;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class TestHiveAccumuloHelper {
-  private static final Logger log = 
Logger.getLogger(TestHiveAccumuloHelper.class);
+  private static final Logger log = 
LoggerFactory.getLogger(TestHiveAccumuloHelper.class);
 
   private HiveAccumuloHelper helper;
 
diff --git 
a/ql/src/java/org/apache/hadoop/hive/ql/udf/esri/HiveGeometryOIHelper.java 
b/ql/src/java/org/apache/hadoop/hive/ql/udf/esri/HiveGeometryOIHelper.java
index c9f49250a66..f20d582dae2 100755
--- a/ql/src/java/org/apache/hadoop/hive/ql/udf/esri/HiveGeometryOIHelper.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/udf/esri/HiveGeometryOIHelper.java
@@ -27,11 +27,12 @@ import 
org.apache.hadoop.hive.serde2.objectinspector.ObjectInspector.Category;
 import org.apache.hadoop.hive.serde2.objectinspector.ObjectInspectorUtils;
 import org.apache.hadoop.hive.serde2.objectinspector.PrimitiveObjectInspector;
 import org.apache.hadoop.io.BytesWritable;
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class HiveGeometryOIHelper {
 
-  static Logger LOG = Logger.getLogger(HiveGeometryOIHelper.class);
+  static Logger LOG = LoggerFactory.getLogger(HiveGeometryOIHelper.class);
 
   private final PrimitiveObjectInspector oi;
   private final int argIndex;
@@ -173,4 +174,4 @@ public class HiveGeometryOIHelper {
 
     return builder.toString();
   }
-}
\ No newline at end of file
+}
diff --git 
a/ql/src/java/org/apache/hadoop/hive/ql/udf/esri/ST_GeometryRelational.java 
b/ql/src/java/org/apache/hadoop/hive/ql/udf/esri/ST_GeometryRelational.java
index d41700bdb13..0bc483552c6 100755
--- a/ql/src/java/org/apache/hadoop/hive/ql/udf/esri/ST_GeometryRelational.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/udf/esri/ST_GeometryRelational.java
@@ -26,14 +26,15 @@ import org.apache.hadoop.hive.ql.metadata.HiveException;
 import org.apache.hadoop.hive.ql.udf.generic.GenericUDF;
 import org.apache.hadoop.hive.serde2.objectinspector.ObjectInspector;
 import 
org.apache.hadoop.hive.serde2.objectinspector.primitive.PrimitiveObjectInspectorFactory;
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * Abstract class that all simple relational tests (contains, touches, ...) 
extend from
  *
  */
 public abstract class ST_GeometryRelational extends GenericUDF {
-  private static final Logger LOG = 
Logger.getLogger(ST_GeometryRelational.class);
+  private static final Logger LOG = 
LoggerFactory.getLogger(ST_GeometryRelational.class);
 
   private static final int NUM_ARGS = 2;
   private static final int GEOM_1 = 0;