zookeeper-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [zookeeper] TisonKun commented on a change in pull request #1049: ZOOKEEPER-3475 Enable Checkstyle configuration on zookeeper-server
Date Mon, 12 Aug 2019 10:48:20 GMT
TisonKun commented on a change in pull request #1049: ZOOKEEPER-3475 Enable Checkstyle configuration
on zookeeper-server
URL: https://github.com/apache/zookeeper/pull/1049#discussion_r312866880
 
 

 ##########
 File path: zookeeper-server/src/main/java/org/apache/zookeeper/client/HostProvider.java
 ##########
 @@ -18,58 +18,60 @@
 
 package org.apache.zookeeper.client;
 
-import org.apache.yetus.audience.InterfaceAudience;
-
 import java.net.InetSocketAddress;
 import java.util.Collection;
+import org.apache.yetus.audience.InterfaceAudience;
 
 /**
  * A set of hosts a ZooKeeper client should connect to.
- * 
- * Classes implementing this interface must guarantee the following:
- * 
- * * Every call to next() returns an InetSocketAddress. So the iterator never
- * ends.
- * 
- * * The size() of a HostProvider may never be zero.
- * 
- * A HostProvider must return resolved InetSocketAddress instances on next() if the next
address is resolvable.
+ *
+ * <p>Classes implementing this interface must guarantee the following:
+ *
+ * <ul>
+ * <li>Every call to next() returns an InetSocketAddress. So the iterator never ends.</li>
+ * <li>The size() of a HostProvider may never be zero.</li>
+ * </ul>
+ *
+ * <p>A HostProvider must return resolved InetSocketAddress instances on next() if
the next address is resolvable.
  * In that case, it's up to the HostProvider, whether it returns the next resolvable address
in the list or return
  * the next one as UnResolved.
- * 
- * Different HostProvider could be imagined:
- * 
- * * A HostProvider that loads the list of Hosts from an URL or from DNS 
- * * A HostProvider that re-resolves the InetSocketAddress after a timeout. 
- * * A HostProvider that prefers nearby hosts.
+ *
+ * <p>ifferent HostProvider could be imagined:
 
 Review comment:
   Nice catch. Fixing...

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message