diff options
| author | David Mollitor <dmollitor@apache.org> | 2020-10-02 11:44:59 -0400 |
|---|---|---|
| committer | Jens Geyer <jensg@apache.org> | 2020-10-04 22:18:13 +0200 |
| commit | d89b427d19a07d556c8a6193e81bd1ed948ac5b5 (patch) | |
| tree | e2a40526680a4085368ad5719830e9a75e067b00 | |
| parent | 5751ddf2ac8df7845c88154a9cc498c46402730d (diff) | |
| download | thrift-d89b427d19a07d556c8a6193e81bd1ed948ac5b5.tar.gz | |
THRIFT-5287: Log When Client Connections are Dropped
Client: java
Patch: David Mollitor
This closes #2253
| -rw-r--r-- | lib/java/src/org/apache/thrift/server/TSimpleServer.java | 1 | ||||
| -rw-r--r-- | lib/java/src/org/apache/thrift/server/TThreadPoolServer.java | 2 |
2 files changed, 3 insertions, 0 deletions
diff --git a/lib/java/src/org/apache/thrift/server/TSimpleServer.java b/lib/java/src/org/apache/thrift/server/TSimpleServer.java index 13501efc6..70cc3d776 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 e2f31d550..467b10d09 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 |