zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hanm <...@git.apache.org>
Subject [GitHub] zookeeper pull request #184: ZOOKEEPER-236: SSL Support for Atomic Broadcast...
Date Fri, 28 Sep 2018 00:03:04 GMT
Github user hanm commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/184#discussion_r220755770
  
    --- Diff: src/java/main/org/apache/zookeeper/server/quorum/Learner.java ---
    @@ -242,9 +247,8 @@ protected void sockConnect(Socket sock, InetSocketAddress addr, int
timeout)
          * @throws InterruptedException
          */
         protected void connectToLeader(InetSocketAddress addr, String hostname)
    -    throws IOException, ConnectException, InterruptedException {
    -        sock = new Socket();        
    -        sock.setSoTimeout(self.tickTime * self.initLimit);
    +    throws IOException, InterruptedException, X509Exception {
    --- End diff --
    
    nit: indentation


---

Mime
View raw message