Use slf4j instead of the apache log4j for logging tables

Author: lbschanno

warehouse/query-core/src/test/java/datawave/helpers/Output.java

@@ -2,8 +2,8 @@
 
 import java.io.PrintStream;
 
-import org.apache.log4j.Level;
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.event.Level;
 
 public interface Output {
 
@@ -15,25 +15,25 @@ default void writeln(String line) {
 
     void flush();
 
-    class ApacheLog4JOutput implements Output {
+    class Slf4jOutput implements Output {
 
         private final StringBuilder sb = new StringBuilder();
         private final Logger log;
         private final Level level;
 
-        public static ApacheLog4JOutput info(Logger log) {
-            return new ApacheLog4JOutput(log, Level.INFO);
+        public static Slf4jOutput info(Logger log) {
+            return new Slf4jOutput(log, Level.INFO);
         }
 
-        public static ApacheLog4JOutput debug(Logger log) {
-            return new ApacheLog4JOutput(log, Level.DEBUG);
+        public static Slf4jOutput debug(Logger log) {
+            return new Slf4jOutput(log, Level.DEBUG);
         }
 
-        public static ApacheLog4JOutput trace(Logger log) {
-            return new ApacheLog4JOutput(log, Level.TRACE);
+        public static Slf4jOutput trace(Logger log) {
+            return new Slf4jOutput(log, Level.TRACE);
         }
 
-        public ApacheLog4JOutput(Logger log, Level level) {
+        public Slf4jOutput(Logger log, Level level) {
             this.log = log;
             this.level = level;
         }
@@ -45,7 +45,7 @@ public void write(String line) {
 
         @Override
         public void flush() {
-            log.log(level, sb.toString());
+            log.atLevel(level).log(sb.toString());
             sb.setLength(0);
         }
     }

warehouse/query-core/src/test/java/datawave/helpers/PrintUtility.java

@@ -18,7 +18,8 @@
 import org.apache.accumulo.core.data.Value;
 import org.apache.accumulo.core.security.Authorizations;
 import org.apache.hadoop.io.WritableUtils;
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import com.clearspring.analytics.stream.cardinality.HyperLogLogPlus;
 
@@ -39,7 +40,7 @@
  */
 public final class PrintUtility {
 
-    private static final Logger log = Logger.getLogger(PrintUtility.class);
+    private static final Logger log = LoggerFactory.getLogger(PrintUtility.class);
 
     private static final DateTimeFormatter ISO_DATE_TIME_MILLIS = DateTimeFormatter.ofPattern("yyyyMMdd-HHmmss").withZone(ZoneId.systemDefault());
 
@@ -359,7 +360,7 @@ public void writeTable(final AccumuloClient client, final Authorizations authori
     public static void printTableToLogDebug(final AccumuloClient client, final Authorizations authorizations, final String tableName, final TableWriter writer)
                     throws TableNotFoundException {
         if (log.isDebugEnabled()) {
-            printTable(client, authorizations, tableName, writer, Output.ApacheLog4JOutput.debug(log));
+            printTable(client, authorizations, tableName, writer, Output.Slf4jOutput.debug(log));
         }
     }
 
@@ -468,7 +469,7 @@ private static String decodeEdgeMetadata(Value value) {
     }
 
     /**
-     * Decode the given value as an compressed and encoded document.
+     * Decode the given value as a compressed and encoded document.
      */
     private static String decodeDocument(Value value) {
         try {