THRIFT-558. java: Replace use of log4j by commons-logging in Java library and generated Java code
This patch switches Thrift to use slf4j instead of log4j.
git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@804349 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/lib/java/ivy.xml b/lib/java/ivy.xml
index 0b1be5d..64203b9 100644
--- a/lib/java/ivy.xml
+++ b/lib/java/ivy.xml
@@ -1,7 +1,8 @@
<ivy-module version="1.0">
<info organisation="jayasoft" module="hello-ivy" />
<dependencies>
- <dependency org="log4j" name="log4j" rev="1.2.15" conf="default->master"/>
+ <dependency org="org.slf4j" name="slf4j-api" rev="1.5.8" conf="* -> *,!sources,!javadoc"/>
+ <dependency org="org.slf4j" name="slf4j-simple" rev="1.5.8" conf="* -> *,!sources,!javadoc"/>
<dependency org="commons-lang" name="commons-lang" rev="2.4" conf="* -> *,!sources,!javadoc"/>
</dependencies>
</ivy-module>
diff --git a/lib/java/src/org/apache/thrift/server/TNonblockingServer.java b/lib/java/src/org/apache/thrift/server/TNonblockingServer.java
index 6d040a7..02fed33 100644
--- a/lib/java/src/org/apache/thrift/server/TNonblockingServer.java
+++ b/lib/java/src/org/apache/thrift/server/TNonblockingServer.java
@@ -30,7 +30,8 @@
import java.util.Iterator;
import java.util.Set;
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import org.apache.thrift.TByteArrayOutputStream;
import org.apache.thrift.TException;
@@ -59,7 +60,7 @@
*/
public class TNonblockingServer extends TServer {
private static final Logger LOGGER =
- Logger.getLogger(TNonblockingServer.class.getName());
+ LoggerFactory.getLogger(TNonblockingServer.class.getName());
// Flag for stopping the server
private volatile boolean stopped_;
diff --git a/lib/java/src/org/apache/thrift/server/TSimpleServer.java b/lib/java/src/org/apache/thrift/server/TSimpleServer.java
index b3ee5ad..585865c 100644
--- a/lib/java/src/org/apache/thrift/server/TSimpleServer.java
+++ b/lib/java/src/org/apache/thrift/server/TSimpleServer.java
@@ -28,7 +28,9 @@
import org.apache.thrift.transport.TTransport;
import org.apache.thrift.transport.TTransportFactory;
import org.apache.thrift.transport.TTransportException;
-import org.apache.log4j.Logger;
+
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
/**
* Simple singlethreaded server for testing.
@@ -36,7 +38,7 @@
*/
public class TSimpleServer extends TServer {
- private static final Logger LOGGER = Logger.getLogger(TSimpleServer.class.getName());
+ private static final Logger LOGGER = LoggerFactory.getLogger(TSimpleServer.class.getName());
private boolean stopped_ = false;
diff --git a/lib/java/src/org/apache/thrift/server/TThreadPoolServer.java b/lib/java/src/org/apache/thrift/server/TThreadPoolServer.java
index ebc5a9b..5b442a0 100644
--- a/lib/java/src/org/apache/thrift/server/TThreadPoolServer.java
+++ b/lib/java/src/org/apache/thrift/server/TThreadPoolServer.java
@@ -29,8 +29,9 @@
import org.apache.thrift.transport.TTransport;
import org.apache.thrift.transport.TTransportException;
import org.apache.thrift.transport.TTransportFactory;
-import org.apache.log4j.Logger;
-import org.apache.log4j.Level;
+
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
@@ -46,7 +47,7 @@
*/
public class TThreadPoolServer extends TServer {
- private static final Logger LOGGER = Logger.getLogger(TThreadPoolServer.class.getName());
+ private static final Logger LOGGER = LoggerFactory.getLogger(TThreadPoolServer.class.getName());
// Executor service for handling client connections
private ExecutorService executorService_;
diff --git a/lib/java/src/org/apache/thrift/transport/TIOStreamTransport.java b/lib/java/src/org/apache/thrift/transport/TIOStreamTransport.java
index 89cdb58..fa5b6d7 100644
--- a/lib/java/src/org/apache/thrift/transport/TIOStreamTransport.java
+++ b/lib/java/src/org/apache/thrift/transport/TIOStreamTransport.java
@@ -19,7 +19,8 @@
package org.apache.thrift.transport;
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import java.io.IOException;
import java.io.InputStream;
@@ -34,7 +35,7 @@
*/
public class TIOStreamTransport extends TTransport {
- private static final Logger LOGGER = Logger.getLogger(TIOStreamTransport.class.getName());
+ private static final Logger LOGGER = LoggerFactory.getLogger(TIOStreamTransport.class.getName());
/** Underlying inputStream */
protected InputStream inputStream_ = null;
diff --git a/lib/java/src/org/apache/thrift/transport/TServerSocket.java b/lib/java/src/org/apache/thrift/transport/TServerSocket.java
index 796cd65..3eca7bc 100644
--- a/lib/java/src/org/apache/thrift/transport/TServerSocket.java
+++ b/lib/java/src/org/apache/thrift/transport/TServerSocket.java
@@ -19,7 +19,8 @@
package org.apache.thrift.transport;
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import java.io.IOException;
import java.net.InetSocketAddress;
@@ -33,7 +34,7 @@
*/
public class TServerSocket extends TServerTransport {
- private static final Logger LOGGER = Logger.getLogger(TServerSocket.class.getName());
+ private static final Logger LOGGER = LoggerFactory.getLogger(TServerSocket.class.getName());
/**
* Underlying serversocket object
diff --git a/lib/java/src/org/apache/thrift/transport/TSocket.java b/lib/java/src/org/apache/thrift/transport/TSocket.java
index cdf1bcc..75ceddc 100644
--- a/lib/java/src/org/apache/thrift/transport/TSocket.java
+++ b/lib/java/src/org/apache/thrift/transport/TSocket.java
@@ -19,7 +19,8 @@
package org.apache.thrift.transport;
-import org.apache.log4j.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
@@ -34,7 +35,7 @@
*/
public class TSocket extends TIOStreamTransport {
- private static final Logger LOGGER = Logger.getLogger(TSocket.class.getName());
+ private static final Logger LOGGER = LoggerFactory.getLogger(TSocket.class.getName());
/**
* Wrapped Socket object