Browse Source

ZOOKEEPER-1052. Findbugs warning in QuorumPeer.ResponderThread.run() (fpj via michim)

git-svn-id: https://svn.apache.org/repos/asf/zookeeper/trunk@1099158 13f79535-47bb-0310-9956-ffa450edef68
Michi Mutsuzaki 14 years ago
parent
commit
db0f374ce7
2 changed files with 6 additions and 2 deletions
  1. 2 0
      CHANGES.txt
  2. 4 2
      src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java

+ 2 - 0
CHANGES.txt

@@ -278,6 +278,8 @@ IMPROVEMENTS:
 
 
   ZOOKEEPER-874. FileTxnSnapLog.restore does not call listener (diogo via fpj)
   ZOOKEEPER-874. FileTxnSnapLog.restore does not call listener (diogo via fpj)
 
 
+  ZOOKEEPER-1052. Findbugs warning in QuorumPeer.ResponderThread.run()  (fpj via michim)
+
 NEW FEATURES:
 NEW FEATURES:
   ZOOKEEPER-729. Java client API to recursively delete a subtree.
   ZOOKEEPER-729. Java client API to recursively delete a subtree.
   (Kay Kay via henry)
   (Kay Kay via henry)

+ 4 - 2
src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java

@@ -313,8 +313,10 @@ public class QuorumPeer extends Thread implements QuorumStats.Provider {
                     }
                     }
                     packet.setLength(b.length);
                     packet.setLength(b.length);
                 }
                 }
-            } catch (Exception e) {
-                LOG.warn("Unexpected exception in ResponderThread",e);
+            } catch (RuntimeException e) {
+                LOG.warn("Unexpected runtime exception in ResponderThread",e);
+            } catch (IOException e) {
+                LOG.warn("Unexpected IO exception in ResponderThread",e);
             } finally {
             } finally {
                 LOG.warn("QuorumPeer responder thread exited");
                 LOG.warn("QuorumPeer responder thread exited");
             }
             }