This is an automated email from the ASF dual-hosted git repository. jking 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 286eee1 THRIFT-4849: Do not Ignore InterruptedException 286eee1 is described below commit 286eee16b147a302ddc7b10740c5e5401ebbec17 Author: Beluga Behr <dam6...@gmail.com> AuthorDate: Wed Apr 17 09:33:58 2019 -0400 THRIFT-4849: Do not Ignore InterruptedException --- lib/java/src/org/apache/thrift/server/TNonblockingServer.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/java/src/org/apache/thrift/server/TNonblockingServer.java b/lib/java/src/org/apache/thrift/server/TNonblockingServer.java index 2358c63..79610b0 100644 --- a/lib/java/src/org/apache/thrift/server/TNonblockingServer.java +++ b/lib/java/src/org/apache/thrift/server/TNonblockingServer.java @@ -86,8 +86,8 @@ public class TNonblockingServer extends AbstractNonblockingServer { try { selectAcceptThread_.join(); } catch (InterruptedException e) { - // for now, just silently ignore. technically this means we'll have less of - // a graceful shutdown as a result. + LOGGER.debug("Interrupted while waiting for accept thread", e); + Thread.currentThread().interrupt(); } }