[thrift] branch master updated: THRIFT-5287: Log When Client Connections are Dropped Client: java Patch: David Mollitor

[jensg]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new d89b427  THRIFT-5287: Log When Client Connections are Dropped Client: 
java Patch: David Mollitor
d89b427 is described below

commit d89b427d19a07d556c8a6193e81bd1ed948ac5b5
Author: David Mollitor <dmolli...@apache.org>
AuthorDate: Fri Oct 2 11:44:59 2020 -0400

    THRIFT-5287: Log When Client Connections are Dropped
    Client: java
    Patch: David Mollitor
    
    This closes #2253
---
 lib/java/src/org/apache/thrift/server/TSimpleServer.java     | 1 +
 lib/java/src/org/apache/thrift/server/TThreadPoolServer.java | 2 ++
 2 files changed, 3 insertions(+)

diff --git a/lib/java/src/org/apache/thrift/server/TSimpleServer.java 
b/lib/java/src/org/apache/thrift/server/TSimpleServer.java
index 13501ef..70cc3d7 100644
--- a/lib/java/src/org/apache/thrift/server/TSimpleServer.java
+++ b/lib/java/src/org/apache/thrift/server/TSimpleServer.java
@@ -82,6 +82,7 @@ public class TSimpleServer extends TServer {
         }
       } catch (TTransportException ttx) {
         // Client died, just move on
+        LOGGER.debug("Client Transportation Exception", ttx);
       } catch (TException tx) {
         if (!stopped_) {
           LOGGER.error("Thrift error occurred during processing of message.", 
tx);
diff --git a/lib/java/src/org/apache/thrift/server/TThreadPoolServer.java 
b/lib/java/src/org/apache/thrift/server/TThreadPoolServer.java
index e2f31d5..467b10d 100644
--- a/lib/java/src/org/apache/thrift/server/TThreadPoolServer.java
+++ b/lib/java/src/org/apache/thrift/server/TThreadPoolServer.java
@@ -315,6 +315,8 @@ public class TThreadPoolServer extends TServer {
             processor.process(inputProtocol, outputProtocol);
         }
       } catch (Exception x) {
+        LOGGER.debug("Error processing request", x);
+
         // We'll usually receive RuntimeException types here
         // Need to unwrap to ascertain real causing exception before we choose 
to ignore
         // Ignore err-logging all transport-level/type exceptions