[thrift] branch master updated: THRIFT-4849: Do not Ignore InterruptedException

[jk...@apache.org]

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 <da...@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();
     }
   }