jiazhai commented on a change in pull request #1012: [Merge Yahoo repo]
CMS-1437: use BookieSocketAddress in DNSToSwitchMapping
URL: https://github.com/apache/bookkeeper/pull/1012#discussion_r162830090
##########
File path:
bookkeeper-server/src/main/java/org/apache/bookkeeper/net/DNSToSwitchMapping.java
##########
@@ -29,25 +29,24 @@
@Beta
public interface DNSToSwitchMapping {
/**
- * Resolves a list of DNS-names/IP-addresses and returns back a list of
+ * Resolves a list of {@link BookieSocketAddress} and returns back a list
of
* switch information (network paths). One-to-one correspondence must be
* maintained between the elements in the lists.
* Consider an element in the argument list - x.y.com. The switch
information
* that is returned must be a network path of the form /foo/rack,
* where / is the root, and 'foo' is the switch where 'rack' is connected.
- * Note the hostname/ip-address is not part of the returned path.
+ * Note the {@link BookieSocketAddress} is not part of the returned path.
* The network topology of the cluster would determine the number of
* components in the network path.
- *
- * <p>If a name cannot be resolved to a rack, the implementation
- * should return {@link NetworkTopology#DEFAULT_REGION_AND_RACK}. This
+ * If a {@link BookieSocketAddress} cannot be resolved to a rack, the
implementation
+ * should return {@link NetworkTopology#DEFAULT_RACK}. This
* is what the bundled implementations do, though it is not a formal
requirement
*
- * @param names the list of hosts to resolve (can be empty)
+ * @param bookieAddressList the list of hosts to resolve (can be empty)
* @return list of resolved network paths.
- * If <i>names</i> is empty, the returned list is also empty
+ * If <i>bookieAddressList</i> is empty, the returned list is also empty
*/
- List<String> resolve(List<String> names);
+ List<String> resolve(List<BookieSocketAddress> bookieAddressList);
Review comment:
Thanks. will do it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
