Jelajahi Sumber

ZOOKEEPER-1342. quorum Listener & LearnerCnxAcceptor are missing thread names (Rakesh R via phunt)

git-svn-id: https://svn.apache.org/repos/asf/zookeeper/trunk@1225059 13f79535-47bb-0310-9956-ffa450edef68
Patrick D. Hunt 13 tahun lalu
induk
melakukan
ff7b4f0615

+ 3 - 0
CHANGES.txt

@@ -155,6 +155,9 @@ IMPROVEMENTS:
   ZOOKEEPER-1326. The CLI commands "delete" and "rmr" are confusing.
   ZOOKEEPER-1326. The CLI commands "delete" and "rmr" are confusing.
   Can we have "rm" + "rmr" instead? (Harsh J via phunt)
   Can we have "rm" + "rmr" instead? (Harsh J via phunt)
 
 
+  ZOOKEEPER-1342. quorum Listener & LearnerCnxAcceptor are missing
+  thread names (Rakesh R via phunt)
+
 Release 3.4.0 - 
 Release 3.4.0 - 
 
 
 Non-backward compatible changes:
 Non-backward compatible changes:

+ 3 - 2
src/java/main/org/apache/zookeeper/server/quorum/Leader.java

@@ -126,7 +126,7 @@ public class Leader {
         }
         }
     }
     }
 
 
-    ServerSocket ss;
+    private final ServerSocket ss;
 
 
     Leader(QuorumPeer self,LeaderZooKeeperServer zk) throws IOException {
     Leader(QuorumPeer self,LeaderZooKeeperServer zk) throws IOException {
         this.self = self;
         this.self = self;
@@ -137,7 +137,7 @@ public class Leader {
                     + self.getQuorumAddress().getPort(), e);
                     + self.getQuorumAddress().getPort(), e);
             throw e;
             throw e;
         }
         }
-        this.zk=zk;
+        this.zk = zk;
     }
     }
 
 
     /**
     /**
@@ -306,6 +306,7 @@ public class Leader {
             // Start thread that waits for connection requests from
             // Start thread that waits for connection requests from
             // new followers.
             // new followers.
             cnxAcceptor = new LearnerCnxAcceptor();
             cnxAcceptor = new LearnerCnxAcceptor();
+            cnxAcceptor.setName("LearnerCnxAcceptor-" + ss.getLocalSocketAddress());
             cnxAcceptor.start();
             cnxAcceptor.start();
 
 
             readyToStart = true;
             readyToStart = true;

+ 1 - 0
src/java/main/org/apache/zookeeper/server/quorum/QuorumCnxManager.java

@@ -149,6 +149,7 @@ public class QuorumCnxManager {
 
 
         // Starts listener thread that waits for connection requests 
         // Starts listener thread that waits for connection requests 
         listener = new Listener();
         listener = new Listener();
+        listener.setName("QuorumPeerListener");
     }
     }
 
 
     /**
     /**