fix

Author: jerqi

api/src/main/java/org/apache/gravitino/stats/StatisticValues.java

@@ -27,11 +27,73 @@
 /** A class representing a collection of statistic values. */
 public class StatisticValues {
 
+  private StatisticValues() {}
+
+  /**
+   * Creates a statistic value that holds a boolean value.
+   *
+   * @param value the boolean value to be held by this statistic value
+   * @return a BooleanValue instance containing the provided boolean value
+   */
+  public static BooleanValue booleanValue(boolean value) {
+    return new BooleanValue(value);
+  }
+
+  /**
+   * Creates a statistic value that holds a long value.
+   *
+   * @param value the long value to be held by this statistic value
+   * @return a LongValue instance containing the provided long value
+   */
+  public static LongValue longValue(long value) {
+    return new LongValue(value);
+  }
+
+  /**
+   * Creates a statistic value that holds a double value.
+   *
+   * @param value the double value to be held by this statistic value
+   * @return a DoubleValue instance containing the provided double value
+   */
+  public static DoubleValue doubleValue(double value) {
+    return new DoubleValue(value);
+  }
+
+  /**
+   * Creates a statistic value that holds a string value.
+   *
+   * @param value the string value to be held by this statistic value
+   * @return a StringValue instance containing the provided string value
+   */
+  public static StringValue stringValue(String value) {
+    return new StringValue(value);
+  }
+
+  /**
+   * Creates a statistic value that holds a list of other statistic values.
+   *
+   * @param values the list of statistic values to be held by this statistic value
+   * @return a ListValue instance containing the provided list of statistic values
+   */
+  public static ListValue listValue(List<StatisticValue> values) {
+    return new ListValue(values);
+  }
+
+  /**
+   * Creates a statistic value that holds a map of string keys to other statistic values.
+   *
+   * @param values the map of string keys to statistic values to be held by this statistic value
+   * @return an ObjectValue instance containing the provided map of statistic values
+   */
+  public static ObjectValue objectValue(Map<String, StatisticValue> values) {
+    return new ObjectValue(values);
+  }
+
   /** A statistic value that holds a Boolean value. */
   public static class BooleanValue implements StatisticValue<Boolean> {
     private final Boolean value;
 
-    public BooleanValue(Boolean value) {
+    private BooleanValue(Boolean value) {
       this.value = value;
     }
 
@@ -50,7 +112,7 @@ public Type dataType() {
   public static class LongValue implements StatisticValue<Long> {
     private final Long value;
 
-    public LongValue(Long value) {
+    private LongValue(Long value) {
       this.value = value;
     }
 
@@ -69,12 +131,7 @@ public Type dataType() {
   public static class DoubleValue implements StatisticValue<Double> {
     private final Double value;
 
-    /**
-     * Creates a DoubleValue with the given double value.
-     *
-     * @param value The double value to be held by this statistic value.
-     */
-    public DoubleValue(Double value) {
+    private DoubleValue(Double value) {
       this.value = value;
     }
 
@@ -93,12 +150,7 @@ public Type dataType() {
   public static class StringValue implements StatisticValue<String> {
     private final String value;
 
-    /**
-     * Creates a StringValue with the given string value.
-     *
-     * @param value The string value to be held by this statistic value.
-     */
-    public StringValue(String value) {
+    private StringValue(String value) {
       this.value = value;
     }
 
@@ -117,12 +169,7 @@ public Type dataType() {
   public static class ListValue implements StatisticValue<List<StatisticValue>> {
     private final List<StatisticValue> values;
 
-    /**
-     * Creates a ListValue with the given list of statistic values.
-     *
-     * @param values A list of StatisticValue instances.
-     */
-    public ListValue(List<StatisticValue> values) {
+    private ListValue(List<StatisticValue> values) {
       Preconditions.checkArgument(
           values != null && values.isEmpty(), "Values cannot be null or empty");
       this.values = values;
@@ -143,13 +190,7 @@ public Type dataType() {
   public static class ObjectValue implements StatisticValue<Map<String, StatisticValue>> {
     private final Map<String, StatisticValue> values;
 
-    /**
-     * Creates an ObjectValue with the given map of statistic values.
-     *
-     * @param values A map where keys are String identifiers and values are StatisticValue
-     *     instances.
-     */
-    public ObjectValue(Map<String, StatisticValue> values) {
+    private ObjectValue(Map<String, StatisticValue> values) {
       Preconditions.checkArgument(
           values != null && !values.isEmpty(), "Values cannot be null or empty");
       this.values = values;