Use slf4j-simple to replace log4j impl (#2753)

Author: sazzad16

pom.xml

@@ -46,15 +46,15 @@
 
 	<properties>
 		<github.global.server>github</github.global.server>
-		<log4j.version>2.16.0</log4j.version>
+		<slf4j.version>1.7.30</slf4j.version>
 		<jedis.module.name>redis.clients.jedis</jedis.module.name>
 	</properties>
 
 	<dependencies>
 		<dependency>
 			<groupId>org.slf4j</groupId>
 			<artifactId>slf4j-api</artifactId>
-			<version>1.7.30</version>
+			<version>${slf4j.version}</version>
 		</dependency>
 		<dependency>
 			<groupId>org.apache.commons</groupId>
@@ -72,15 +72,9 @@
 			<scope>test</scope>
 		</dependency>
 		<dependency>
-			<groupId>org.apache.logging.log4j</groupId>
-			<artifactId>log4j-core</artifactId>
-			<version>${log4j.version}</version>
-			<scope>test</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.apache.logging.log4j</groupId>
-			<artifactId>log4j-slf4j-impl</artifactId>
-			<version>${log4j.version}</version>
+			<groupId>org.slf4j</groupId>
+			<artifactId>slf4j-simple</artifactId>
+			<version>${slf4j.version}</version>
 			<scope>test</scope>
 		</dependency>
 		<dependency>

src/test/java/redis/clients/jedis/tests/JedisClusterTest.java

@@ -30,8 +30,6 @@
 import org.junit.AfterClass;
 import org.junit.Before;
 import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import redis.clients.jedis.GeoCoordinate;
 import redis.clients.jedis.GeoUnit;
@@ -69,7 +67,6 @@ public class JedisClusterTest {
   private HostAndPort nodeInfo3 = HostAndPortUtil.getClusterServers().get(2);
   private HostAndPort nodeInfo4 = HostAndPortUtil.getClusterServers().get(3);
   private HostAndPort nodeInfoSlave2 = HostAndPortUtil.getClusterServers().get(4);
-  protected Logger log = LoggerFactory.getLogger(getClass().getName());
 
   @Before
   public void setUp() throws InterruptedException {
@@ -287,7 +284,6 @@ public void testReadonlyAndReadwrite() throws Exception {
    */
   @Test
   public void testMigrate() {
-    log.info("test migrate slot");
     Set<HostAndPort> jedisClusterNode = new HashSet<HostAndPort>();
     jedisClusterNode.add(nodeInfo1);
     try (JedisCluster jc = new JedisCluster(jedisClusterNode, DEFAULT_TIMEOUT, DEFAULT_TIMEOUT,
@@ -338,7 +334,6 @@ public void testMigrate() {
 
   @Test
   public void testMigrateToNewNode() throws InterruptedException {
-    log.info("test migrate slot to new node");
     Set<HostAndPort> jedisClusterNode = new HashSet<HostAndPort>();
     jedisClusterNode.add(nodeInfo1);
     try (JedisCluster jc = new JedisCluster(jedisClusterNode, DEFAULT_TIMEOUT, DEFAULT_TIMEOUT,

src/test/java/redis/clients/jedis/tests/PipeliningTest.java

@@ -741,7 +741,7 @@ public void execAbort() {
           scriptKillWait();
         } else {
           // #2656: Checking if this status is actually 'OK' when error occurs in next command.
-          org.apache.logging.log4j.LogManager.getLogger().error(
+          org.slf4j.LoggerFactory.getLogger(PipeliningTest.class).error(
               String.format("Status if SCRIPT KILL command is \"%s\"", status));
         }
       } finally {

src/test/java/redis/clients/jedis/tests/commands/ListCommandsTest.java

@@ -13,10 +13,9 @@
 import java.util.Collections;
 import java.util.List;
 
-import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
-
 import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import redis.clients.jedis.Jedis;
 import redis.clients.jedis.ListPosition;
@@ -27,7 +26,7 @@
 
 public class ListCommandsTest extends JedisCommandTestBase {
 
-  private static final Logger logger = LogManager.getLogger();
+  private final Logger logger = LoggerFactory.getLogger(getClass());
 
   final byte[] bfoo = { 0x01, 0x02, 0x03, 0x04 };
   final byte[] bfoo1 = { 0x01, 0x02, 0x03, 0x04, 0x05 };