javadoc

ambry-clustermap/src/main/java/com/github/ambry/clustermap/AccountStorageStatsIterator.java

@@ -10,6 +10,9 @@
 import org.slf4j.LoggerFactory;
 
 
+/**
+ * This is an iterator for account storage stats fetched from MySQL
+ */
 public class AccountStorageStatsIterator implements Iterator<Pair<String, HostAccountStorageStatsWrapper>> {
 
   private static final Logger logger = LoggerFactory.getLogger(AccountStorageStatsIterator.class);

ambry-clustermap/src/main/java/com/github/ambry/clustermap/PartitionClassStorageStatsIterator.java

@@ -11,7 +11,9 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-
+/**
+ * This is an iterator for partition storage stats fetched from MySQL
+ */
 public class PartitionClassStorageStatsIterator implements
                                                 Iterator<Pair<String, HostPartitionClassStorageStatsWrapper>> {
 
@@ -47,5 +49,4 @@ public Pair<String, HostPartitionClassStorageStatsWrapper> next() {
       throw new RuntimeException(e);
     }
   }
-
-}
+}

ambry-clustermap/src/main/java/com/github/ambry/clustermap/TaskUtils.java

@@ -5,6 +5,13 @@
 
 public class TaskUtils {
 
+  /**
+   * Given an instance name of the form hostname_port, returns a pair of <hostname, port>.
+   * Given an instance name of the form hostname, returns a pair of <hostname, defaultPort>.
+   * @param instanceName Name of the instance machine
+   * @param defaultPort Default port used by machines
+   * @return Pair of <hostname, port>
+   */
   protected static Pair<String, Integer> getHostNameAndPort(String instanceName, int defaultPort) {
     String hostname = instanceName;
     int port = defaultPort;