From 7b96b2249a43ae75b48e0aba7e8beffc67b32d93 Mon Sep 17 00:00:00 2001 From: Roger Meier Date: Wed, 23 Jan 2013 20:18:58 +0100 Subject: [PATCH] THRIFT-1805 Thrift should not swallow ALL exceptions Patch: Diwaker Gupta --- lib/java/src/org/apache/thrift/ProcessFunction.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/java/src/org/apache/thrift/ProcessFunction.java b/lib/java/src/org/apache/thrift/ProcessFunction.java index 2ba35279..19c8f145 100644 --- a/lib/java/src/org/apache/thrift/ProcessFunction.java +++ b/lib/java/src/org/apache/thrift/ProcessFunction.java @@ -37,8 +37,8 @@ public abstract class ProcessFunction { try { result = getResult(iface, args); - } catch(Throwable th) { - LOGGER.error("Internal error processing " + getMethodName(), th); + } catch(TException tex) { + LOGGER.error("Internal error processing " + getMethodName(), tex); TApplicationException x = new TApplicationException(TApplicationException.INTERNAL_ERROR, "Internal error processing " + getMethodName()); oprot.writeMessageBegin(new TMessage(getMethodName(), TMessageType.EXCEPTION, seqid)); -- 2.17.1