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

Author: ngsg

accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/TestHiveAccumuloHelper.java

@@ -31,14 +31,15 @@
 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;
 

ql/src/java/org/apache/hadoop/hive/ql/udf/esri/HiveGeometryOIHelper.java

@@ -27,11 +27,12 @@
 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 String toString() {
 
     return builder.toString();
   }
-}
+}

ql/src/java/org/apache/hadoop/hive/ql/udf/esri/ST_GeometryRelational.java

@@ -26,14 +26,15 @@
 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;