Merge pull request #657 from apache/main21-2

Main21-2 to main

Author: leerho

pom.xml

@@ -190,7 +190,7 @@ under the License.
               <configuration>
                 <rules>
                   <requireJavaVersion>
-                    <version>${java.version}</version>
+                    <version>[21,)</version> <!-- java.version -->
                   </requireJavaVersion>
                   <requireMavenVersion>
                     <version>[${maven.version},4.0.0)</version>
@@ -237,9 +237,10 @@ under the License.
           <configuration>
             <docfilessubdirs>true</docfilessubdirs>
             <show>public</show>
-            <additionalOptions>
-              <additionalOption>${jvm-ffm-flag}</additionalOption>
-            </additionalOptions>
+            <doclint>all,-missing</doclint>
+            <additionalJOptions>
+              <additionalJOption>${jvm-ffm-flag}</additionalJOption>
+            </additionalJOptions>
           </configuration>
           <executions>
             <execution>

src/main/java/org/apache/datasketches/common/Family.java

@@ -194,7 +194,7 @@ public int getID() {
   }
 
   /**
-   *
+   * Checks the given Family ID
    * @param id the given id, a value < 128.
    */
   public void checkFamilyID(final int id) {

src/main/java/org/apache/datasketches/cpc/MergingValidation.java

@@ -56,7 +56,7 @@ public class MergingValidation {
   private PrintWriter printWriter;
 
   /**
-   *
+   * Constructor
    * @param lgMinK lgMinK
    * @param lgMaxK lgMaxK
    * @param lgMulK lgMulK
@@ -79,7 +79,7 @@ public MergingValidation(final int lgMinK, final int lgMaxK, final int lgMulK, f
   }
 
   /**
-   *
+   * Start the merging validation process
    */
   public void start() {
     printf(hfmt, (Object[]) hStrArr); //print header

src/main/java/org/apache/datasketches/cpc/QuickMergingValidation.java

@@ -54,7 +54,7 @@ public class QuickMergingValidation {
   private PrintWriter printWriter;
 
   /**
-   *
+   * Constructor
    * @param lgMinK lgMinK
    * @param lgMaxK lgMaxK
    * @param incLgK incLgK
@@ -73,7 +73,7 @@ public QuickMergingValidation(final int lgMinK, final int lgMaxK, final int incL
   }
 
   /**
-   *
+   * Start the Quick Merging Validation process
    */
   public void start() {
     printf(hfmt, (Object[]) hStrArr); //print header

src/main/java/org/apache/datasketches/cpc/StreamingValidation.java

@@ -57,7 +57,7 @@ public class StreamingValidation {
   private BitMatrix matrix = null;
 
   /**
-   *
+   * Constructor
    * @param lgMinK lgMinK
    * @param lgMaxK lgMaxK
    * @param trials trials
@@ -78,7 +78,7 @@ public StreamingValidation(final int lgMinK, final int lgMaxK, final int trials,
   }
 
   /**
-   *
+   * Start the streaming validation process
    */
   public void start() {
     printf(hfmt, (Object[]) hStrArr);

src/main/java/org/apache/datasketches/cpc/TestUtil.java

@@ -25,6 +25,7 @@
 import static org.apache.datasketches.cpc.RuntimeAsserts.rtAssertEquals;
 
 /**
+ * Utility methods for Test
  * @author Lee Rhodes
  */
 public class TestUtil {

src/main/java/org/apache/datasketches/fdt/Group.java

@@ -66,41 +66,49 @@ public Group init(final String priKey, final int count, final double estimate, f
   }
 
   /**
-   * @return priKey of type T
+   * Gets the primary key of type String
+   * @return priKey of type String
    */
   public String getPrimaryKey() { return priKey; }
 
   /**
+   * Returns the count
    * @return the count
    */
   public int getCount() { return count; }
 
   /**
+   * Returns the estimate
    * @return the estimate
    */
   public double getEstimate() { return est; }
 
   /**
+   * Returns the upper bound
    * @return the upper bound
    */
   public double getUpperBound() { return ub; }
 
   /**
+   * Returns the lower bound
    * @return the lower bound
    */
   public double getLowerBound() { return lb; }
 
   /**
+   * Returns the fraction for this group
    * @return the fraction for this group
    */
   public double getFraction() { return fraction; }
 
   /**
+   * Returns the RSE
    * @return the RSE
    */
   public double getRse() { return rse; }
 
   /**
+   * Returns the descriptive header
    * @return the descriptive header
    */
   public String getHeader() {
@@ -112,9 +120,6 @@ public String toString() {
     return String.format(fmt, count, est, ub, lb, fraction, rse, priKey);
   }
 
-  /**
-   * @param that The Group to compare to
-   */
   @Override
   public int compareTo(final Group that) {
     return that.count - count; //decreasing
@@ -133,4 +138,3 @@ public int hashCode() {
   }
 
 }
-

src/main/java/org/apache/datasketches/frequencies/ItemsSketch.java

@@ -397,6 +397,7 @@ public Row<T>[] getFrequentItems(final ErrorType errorType) {
   }
 
   /**
+   * Returns an upper bound on the maximum error of getEstimate(item) for any item.
    * @return An upper bound on the maximum error of getEstimate(item) for any item.
    * This is equivalent to the maximum distance between the upper bound and the lower bound
    * for any item.
@@ -415,6 +416,7 @@ public int getMaximumMapCapacity() {
   }
 
   /**
+   * Returns the number of active items in the sketch.
    * @return the number of active items in the sketch.
    */
   public int getNumActiveItems() {
@@ -624,26 +626,31 @@ public static class Row<T> implements Comparable<Row<T>> {
     }
 
     /**
+     * Returns an item of type T
      * @return item of type T
      */
     public T getItem() { return item; }
 
     /**
+     * Returns the estimate
      * @return the estimate
      */
     public long getEstimate() { return est; }
 
     /**
+     * Returns the upper bound
      * @return the upper bound
      */
     public long getUpperBound() { return ub; }
 
     /**
+     * Returns the lower bound
      * @return return the lower bound
      */
     public long getLowerBound() { return lb; }
 
     /**
+     * Returns the descriptive row header
      * @return the descriptive row header
      */
     public static String getRowHeader() {

src/main/java/org/apache/datasketches/frequencies/LongsSketch.java

@@ -439,6 +439,7 @@ public Row[] getFrequentItems(final ErrorType errorType) {
   }
 
   /**
+   * Returns an upper bound on the maximum error of getEstimate(item) for any item.
    * @return An upper bound on the maximum error of getEstimate(item) for any item.
    * This is equivalent to the maximum distance between the upper bound and the lower bound
    * for any item.
@@ -457,6 +458,7 @@ public int getMaximumMapCapacity() {
   }
 
   /**
+   * Returns the number of active items in the sketch.
    * @return the number of active items in the sketch.
    */
   public int getNumActiveItems() {
@@ -693,26 +695,31 @@ public static class Row implements Comparable<Row> {
     }
 
     /**
-     * @return item of type T
+     * Returns item of type long
+     * @return item of type long
      */
     public long getItem() { return item; }
 
     /**
+     * Returns the estimate
      * @return the estimate
      */
     public long getEstimate() { return est; }
 
     /**
+     * Returns the upper bound
      * @return the upper bound
      */
     public long getUpperBound() { return ub; }
 
     /**
+     * Returns the lower bound
      * @return return the lower bound
      */
     public long getLowerBound() { return lb; }
 
     /**
+     * Returns the descriptive row header
      * @return the descriptive row header
      */
     public static String getRowHeader() {

src/main/java/org/apache/datasketches/quantiles/UpdateDoublesSketch.java

@@ -23,6 +23,7 @@
 import org.apache.datasketches.memory.WritableMemory;
 
 /**
+ * Extends DoubleSketch
  * @author Jon Malkin
  */
 public abstract class UpdateDoublesSketch extends DoublesSketch {
@@ -61,6 +62,7 @@ public static UpdateDoublesSketch heapify(final Memory srcMem) {
   }
 
   /**
+   * Returns a CompactDoublesSketch of this class
    * @return a CompactDoublesSketch of this class
    */
   public CompactDoublesSketch compact() {