diff options
author | Bryan Duxbury <bryanduxbury@apache.org> | 2011-11-02 22:40:01 +0000 |
---|---|---|
committer | Bryan Duxbury <bryanduxbury@apache.org> | 2011-11-02 22:40:01 +0000 |
commit | e7bdf70daba05e2fa704a3562eb924573342777b (patch) | |
tree | 24e06550e4dd544d5fbd91eb244911c12baf807f | |
parent | 61188a47975da49fd6f28789cf2b45590766fa96 (diff) | |
download | thrift-e7bdf70daba05e2fa704a3562eb924573342777b.tar.gz |
THRIFT-1419. java: AbstractNonBlockingServer does not catch errors when invoking the processor
This patch changes the invoke() method to catch Throwables instead of just Exceptions.
Patch: Thomas Kielbus"
git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1196848 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r-- | lib/java/src/org/apache/thrift/server/AbstractNonblockingServer.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/java/src/org/apache/thrift/server/AbstractNonblockingServer.java b/lib/java/src/org/apache/thrift/server/AbstractNonblockingServer.java index 2bd74fa84..65d8738ac 100644 --- a/lib/java/src/org/apache/thrift/server/AbstractNonblockingServer.java +++ b/lib/java/src/org/apache/thrift/server/AbstractNonblockingServer.java @@ -479,10 +479,10 @@ public abstract class AbstractNonblockingServer extends TServer { return; } catch (TException te) { LOGGER.warn("Exception while invoking!", te); - } catch (Exception e) { - LOGGER.error("Unexpected exception while invoking!", e); + } catch (Throwable t) { + LOGGER.error("Unexpected throwable while invoking!", t); } - // This will only be reached when there is an exception. + // This will only be reached when there is a throwable. state_ = FrameBufferState.AWAITING_CLOSE; requestSelectInterestChange(); } |