This is an automated email from the ASF dual-hosted git repository.

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 1fdeadd6ef [Minor] Make all constants in  public access (#15605)
1fdeadd6ef is described below

commit 1fdeadd6ef9ad3c6ca51f198ac0d4f922e92a0b7
Author: Xiaotian (Jackie) Jiang <17555551+jackie-ji...@users.noreply.github.com>
AuthorDate: Mon Apr 21 15:35:27 2025 -0600

    [Minor] Make all constants in  public access (#15605)
---
 .../apache/pinot/controller/ControllerConf.java    | 132 ++++++++++-----------
 1 file changed, 66 insertions(+), 66 deletions(-)

diff --git 
a/pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java
 
b/pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java
index afb17232ca..510b8725d0 100644
--- 
a/pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java
+++ 
b/pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java
@@ -257,50 +257,50 @@ public class ControllerConf extends PinotConfiguration {
     public static final int MAX_INITIAL_DELAY_IN_SECONDS = 300;
     public static final int DEFAULT_SPLIT_COMMIT_TMP_SEGMENT_LIFETIME_SECOND = 
60 * 60; // 1 Hour.
 
-    private static final Random RANDOM = new Random();
+    public static final Random RANDOM = new Random();
 
-    private static long getRandomInitialDelayInSeconds() {
+    public static long getRandomInitialDelayInSeconds() {
       return MIN_INITIAL_DELAY_IN_SECONDS + 
RANDOM.nextInt(MAX_INITIAL_DELAY_IN_SECONDS - MIN_INITIAL_DELAY_IN_SECONDS);
     }
 
     // Default values
-    private static final int DEFAULT_RETENTION_MANAGER_FREQUENCY_IN_SECONDS = 
6 * 60 * 60; // 6 Hours.
-    private static final int 
DEFAULT_OFFLINE_SEGMENT_INTERVAL_CHECKER_FREQUENCY_IN_SECONDS = 24 * 60 * 60; 
// 24 Hours.
-    private static final int 
DEFAULT_REALTIME_SEGMENT_VALIDATION_FREQUENCY_IN_SECONDS = 60 * 60; // 1 Hour.
-    private static final int 
DEFAULT_BROKER_RESOURCE_VALIDATION_FREQUENCY_IN_SECONDS = 60 * 60; // 1 Hour.
-    private static final int DEFAULT_STATUS_CHECKER_FREQUENCY_IN_SECONDS = 5 * 
60; // 5 minutes
-    private static final int DEFAULT_REBALANCE_CHECKER_FREQUENCY_IN_SECONDS = 
5 * 60; // 5 minutes
-    private static final int DEFAULT_TASK_METRICS_EMITTER_FREQUENCY_IN_SECONDS 
= 5 * 60; // 5 minutes
-    private static final int 
DEFAULT_STATUS_CONTROLLER_WAIT_FOR_PUSH_TIME_IN_SECONDS = 10 * 60; // 10 minutes
-    private static final int DEFAULT_TASK_MANAGER_FREQUENCY_IN_SECONDS = -1; 
// Disabled
+    public static final int DEFAULT_RETENTION_MANAGER_FREQUENCY_IN_SECONDS = 6 
* 60 * 60; // 6 Hours.
+    public static final int 
DEFAULT_OFFLINE_SEGMENT_INTERVAL_CHECKER_FREQUENCY_IN_SECONDS = 24 * 60 * 60; 
// 24 Hours.
+    public static final int 
DEFAULT_REALTIME_SEGMENT_VALIDATION_FREQUENCY_IN_SECONDS = 60 * 60; // 1 Hour.
+    public static final int 
DEFAULT_BROKER_RESOURCE_VALIDATION_FREQUENCY_IN_SECONDS = 60 * 60; // 1 Hour.
+    public static final int DEFAULT_STATUS_CHECKER_FREQUENCY_IN_SECONDS = 5 * 
60; // 5 minutes
+    public static final int DEFAULT_REBALANCE_CHECKER_FREQUENCY_IN_SECONDS = 5 
* 60; // 5 minutes
+    public static final int DEFAULT_TASK_METRICS_EMITTER_FREQUENCY_IN_SECONDS 
= 5 * 60; // 5 minutes
+    public static final int 
DEFAULT_STATUS_CONTROLLER_WAIT_FOR_PUSH_TIME_IN_SECONDS = 10 * 60; // 10 minutes
+    public static final int DEFAULT_TASK_MANAGER_FREQUENCY_IN_SECONDS = -1; // 
Disabled
     @Deprecated
-    private static final int 
DEFAULT_MINION_INSTANCES_CLEANUP_TASK_FREQUENCY_IN_SECONDS = 60 * 60; // 1 Hour.
+    public static final int 
DEFAULT_MINION_INSTANCES_CLEANUP_TASK_FREQUENCY_IN_SECONDS = 60 * 60; // 1 Hour.
     @Deprecated
-    private static final int 
DEFAULT_MINION_INSTANCES_CLEANUP_TASK_MIN_OFFLINE_TIME_BEFORE_DELETION_IN_SECONDS
 =
+    public static final int 
DEFAULT_MINION_INSTANCES_CLEANUP_TASK_MIN_OFFLINE_TIME_BEFORE_DELETION_IN_SECONDS
 =
         60 * 60; // 1 Hour.
 
-    private static final int 
DEFAULT_SEGMENT_LEVEL_VALIDATION_INTERVAL_IN_SECONDS = 24 * 60 * 60;
-    private static final int DEFAULT_SEGMENT_RELOCATOR_FREQUENCY_IN_SECONDS = 
60 * 60;
+    public static final int 
DEFAULT_SEGMENT_LEVEL_VALIDATION_INTERVAL_IN_SECONDS = 24 * 60 * 60;
+    public static final int DEFAULT_SEGMENT_RELOCATOR_FREQUENCY_IN_SECONDS = 
60 * 60;
 
     // Realtime Consumer Monitor
-    private static final String RT_CONSUMER_MONITOR_FREQUENCY_PERIOD =
+    public static final String RT_CONSUMER_MONITOR_FREQUENCY_PERIOD =
         "controller.realtimeConsumerMonitor.frequencyPeriod";
-    private static final String RT_CONSUMER_MONITOR_INITIAL_DELAY_IN_SECONDS =
+    public static final String RT_CONSUMER_MONITOR_INITIAL_DELAY_IN_SECONDS =
         "controller.realtimeConsumerMonitor.initialDelayInSeconds";
 
-    private static final int DEFAULT_RT_CONSUMER_MONITOR_FREQUENCY_IN_SECONDS 
= -1; // Disabled by default
+    public static final int DEFAULT_RT_CONSUMER_MONITOR_FREQUENCY_IN_SECONDS = 
-1; // Disabled by default
   }
 
-  private static final String SERVER_ADMIN_REQUEST_TIMEOUT_SECONDS = 
"server.request.timeoutSeconds";
-  private static final String MINION_ADMIN_REQUEST_TIMEOUT_SECONDS = 
"minion.request.timeoutSeconds";
-  private static final String SEGMENT_COMMIT_TIMEOUT_SECONDS = 
"controller.realtime.segment.commit.timeoutSeconds";
-  private static final String CONTROLLER_EXECUTOR_NUM_THREADS = 
"controller.executor.numThreads";
+  public static final String SERVER_ADMIN_REQUEST_TIMEOUT_SECONDS = 
"server.request.timeoutSeconds";
+  public static final String MINION_ADMIN_REQUEST_TIMEOUT_SECONDS = 
"minion.request.timeoutSeconds";
+  public static final String SEGMENT_COMMIT_TIMEOUT_SECONDS = 
"controller.realtime.segment.commit.timeoutSeconds";
+  public static final String CONTROLLER_EXECUTOR_NUM_THREADS = 
"controller.executor.numThreads";
   public static final String CONTROLLER_EXECUTOR_REBALANCE_NUM_THREADS = 
"controller.executor.rebalance.numThreads";
 
-  private static final String DELETED_SEGMENTS_RETENTION_IN_DAYS = 
"controller.deleted.segments.retentionInDays";
+  public static final String DELETED_SEGMENTS_RETENTION_IN_DAYS = 
"controller.deleted.segments.retentionInDays";
   public static final String TABLE_MIN_REPLICAS = "table.minReplicas";
-  private static final String JERSEY_ADMIN_API_PORT = "jersey.admin.api.port";
-  private static final String JERSEY_ADMIN_IS_PRIMARY = 
"jersey.admin.isprimary";
+  public static final String JERSEY_ADMIN_API_PORT = "jersey.admin.api.port";
+  public static final String JERSEY_ADMIN_IS_PRIMARY = 
"jersey.admin.isprimary";
   public static final String ACCESS_CONTROL_FACTORY_CLASS = 
"controller.admin.access.control.factory.class";
   public static final String ACCESS_CONTROL_USERNAME = 
"access.control.init.username";
   public static final String ACCESS_CONTROL_PASSWORD = 
"access.control.init.password";
@@ -308,61 +308,61 @@ public class ControllerConf extends PinotConfiguration {
   public static final String REBALANCE_PRE_CHECKER_CLASS = 
"controller.rebalance.pre.checker.class";
   // Amount of the time the segment can take from the beginning of upload to 
the end of upload. Used when parallel push
   // protection is enabled. If the upload does not finish within the timeout, 
next upload can override the previous one.
-  private static final String SEGMENT_UPLOAD_TIMEOUT_IN_MILLIS = 
"controller.segment.upload.timeoutInMillis";
-  private static final String REALTIME_SEGMENT_METADATA_COMMIT_NUMLOCKS =
+  public static final String SEGMENT_UPLOAD_TIMEOUT_IN_MILLIS = 
"controller.segment.upload.timeoutInMillis";
+  public static final String REALTIME_SEGMENT_METADATA_COMMIT_NUMLOCKS =
       "controller.realtime.segment.metadata.commit.numLocks";
-  private static final String ENABLE_STORAGE_QUOTA_CHECK = 
"controller.enable.storage.quota.check";
-  private static final String REBALANCE_DISK_UTILIZATION_THRESHOLD = 
"controller.rebalance.disk.utilization.threshold";
-  private static final String DISK_UTILIZATION_THRESHOLD = 
"controller.disk.utilization.threshold"; // 0 < threshold < 1
-  private static final String DISK_UTILIZATION_CHECK_TIMEOUT_MS = 
"controller.disk.utilization.check.timeoutMs";
-  private static final String DISK_UTILIZATION_PATH = 
"controller.disk.utilization.path";
-  private static final String ENABLE_RESOURCE_UTILIZATION_CHECK = 
"controller.enable.resource.utilization.check";
+  public static final String ENABLE_STORAGE_QUOTA_CHECK = 
"controller.enable.storage.quota.check";
+  public static final String REBALANCE_DISK_UTILIZATION_THRESHOLD = 
"controller.rebalance.disk.utilization.threshold";
+  public static final String DISK_UTILIZATION_THRESHOLD = 
"controller.disk.utilization.threshold"; // 0 < threshold < 1
+  public static final String DISK_UTILIZATION_CHECK_TIMEOUT_MS = 
"controller.disk.utilization.check.timeoutMs";
+  public static final String DISK_UTILIZATION_PATH = 
"controller.disk.utilization.path";
+  public static final String ENABLE_RESOURCE_UTILIZATION_CHECK = 
"controller.enable.resource.utilization.check";
   public static final String RESOURCE_UTILIZATION_CHECKER_INITIAL_DELAY =
       "controller.resource.utilization.checker.initial.delay";
-  private static final String RESOURCE_UTILIZATION_CHECKER_FREQUENCY =
+  public static final String RESOURCE_UTILIZATION_CHECKER_FREQUENCY =
       "controller.resource.utilization.checker.frequency";
-  private static final String ENABLE_BATCH_MESSAGE_MODE = 
"controller.enable.batch.message.mode";
+  public static final String ENABLE_BATCH_MESSAGE_MODE = 
"controller.enable.batch.message.mode";
   public static final String ENABLE_HYBRID_TABLE_RETENTION_STRATEGY =
       "controller.enable.hybrid.table.retention.strategy";
   public static final String DIM_TABLE_MAX_SIZE = 
"controller.dimTable.maxSize";
 
   // Defines the kind of storage and the underlying PinotFS implementation
-  private static final String PINOT_FS_FACTORY_CLASS_LOCAL = 
"controller.storage.factory.class.file";
-
-  private static final int DEFAULT_SERVER_ADMIN_REQUEST_TIMEOUT_SECONDS = 30;
-  private static final int DEFAULT_MINION_ADMIN_REQUEST_TIMEOUT_SECONDS = 30;
-  private static final int DEFAULT_DELETED_SEGMENTS_RETENTION_IN_DAYS = 7;
-  private static final int DEFAULT_TABLE_MIN_REPLICAS = 1;
-  private static final int DEFAULT_JERSEY_ADMIN_PORT = 21000;
-  private static final String DEFAULT_ACCESS_CONTROL_FACTORY_CLASS =
+  public static final String PINOT_FS_FACTORY_CLASS_LOCAL = 
"controller.storage.factory.class.file";
+
+  public static final int DEFAULT_SERVER_ADMIN_REQUEST_TIMEOUT_SECONDS = 30;
+  public static final int DEFAULT_MINION_ADMIN_REQUEST_TIMEOUT_SECONDS = 30;
+  public static final int DEFAULT_DELETED_SEGMENTS_RETENTION_IN_DAYS = 7;
+  public static final int DEFAULT_TABLE_MIN_REPLICAS = 1;
+  public static final int DEFAULT_JERSEY_ADMIN_PORT = 21000;
+  public static final String DEFAULT_ACCESS_CONTROL_FACTORY_CLASS =
       "org.apache.pinot.controller.api.access.AllowAllAccessFactory";
-  private static final String DEFAULT_ACCESS_CONTROL_USERNAME = "admin";
-  private static final String DEFAULT_ACCESS_CONTROL_PASSWORD = "admin";
-  private static final String DEFAULT_LINEAGE_MANAGER =
+  public static final String DEFAULT_ACCESS_CONTROL_USERNAME = "admin";
+  public static final String DEFAULT_ACCESS_CONTROL_PASSWORD = "admin";
+  public static final String DEFAULT_LINEAGE_MANAGER =
       "org.apache.pinot.controller.helix.core.lineage.DefaultLineageManager";
-  private static final String DEFAULT_REBALANCE_PRE_CHECKER =
+  public static final String DEFAULT_REBALANCE_PRE_CHECKER =
       
"org.apache.pinot.controller.helix.core.rebalance.DefaultRebalancePreChecker";
-  private static final long DEFAULT_SEGMENT_UPLOAD_TIMEOUT_IN_MILLIS = 
600_000L; // 10 minutes
-  private static final int DEFAULT_MIN_NUM_CHARS_IN_IS_TO_TURN_ON_COMPRESSION 
= -1;
-  private static final int DEFAULT_REALTIME_SEGMENT_METADATA_COMMIT_NUMLOCKS = 
64;
-  private static final boolean DEFAULT_ENABLE_STORAGE_QUOTA_CHECK = true;
-  private static final double DEFAULT_REBALANCE_DISK_UTILIZATION_THRESHOLD = 
0.9;
-  private static final double DEFAULT_DISK_UTILIZATION_THRESHOLD = 0.95;
-  private static final int DEFAULT_DISK_UTILIZATION_CHECK_TIMEOUT_MS = 30_000;
-  private static final String DEFAULT_DISK_UTILIZATION_PATH = 
"/home/pinot/data";
-  private static final boolean DEFAULT_ENABLE_RESOURCE_UTILIZATION_CHECK = 
false;
-  private static final long DEFAULT_RESOURCE_UTILIZATION_CHECKER_INITIAL_DELAY 
= 300L; // 5 minutes
-  private static final long DEFAULT_RESOURCE_UTILIZATION_CHECKER_FREQUENCY = 
300L; // 5 minutes
-  private static final boolean DEFAULT_ENABLE_BATCH_MESSAGE_MODE = false;
-  private static final boolean DEFAULT_ENABLE_HYBRID_TABLE_RETENTION_STRATEGY 
= false;
-  private static final String DEFAULT_CONTROLLER_MODE = 
ControllerMode.DUAL.name();
-  private static final String 
DEFAULT_LEAD_CONTROLLER_RESOURCE_REBALANCE_STRATEGY =
+  public static final long DEFAULT_SEGMENT_UPLOAD_TIMEOUT_IN_MILLIS = 
600_000L; // 10 minutes
+  public static final int DEFAULT_MIN_NUM_CHARS_IN_IS_TO_TURN_ON_COMPRESSION = 
-1;
+  public static final int DEFAULT_REALTIME_SEGMENT_METADATA_COMMIT_NUMLOCKS = 
64;
+  public static final boolean DEFAULT_ENABLE_STORAGE_QUOTA_CHECK = true;
+  public static final double DEFAULT_REBALANCE_DISK_UTILIZATION_THRESHOLD = 
0.9;
+  public static final double DEFAULT_DISK_UTILIZATION_THRESHOLD = 0.95;
+  public static final int DEFAULT_DISK_UTILIZATION_CHECK_TIMEOUT_MS = 30_000;
+  public static final String DEFAULT_DISK_UTILIZATION_PATH = 
"/home/pinot/data";
+  public static final boolean DEFAULT_ENABLE_RESOURCE_UTILIZATION_CHECK = 
false;
+  public static final long DEFAULT_RESOURCE_UTILIZATION_CHECKER_INITIAL_DELAY 
= 300L; // 5 minutes
+  public static final long DEFAULT_RESOURCE_UTILIZATION_CHECKER_FREQUENCY = 
300L; // 5 minutes
+  public static final boolean DEFAULT_ENABLE_BATCH_MESSAGE_MODE = false;
+  public static final boolean DEFAULT_ENABLE_HYBRID_TABLE_RETENTION_STRATEGY = 
false;
+  public static final String DEFAULT_CONTROLLER_MODE = 
ControllerMode.DUAL.name();
+  public static final String 
DEFAULT_LEAD_CONTROLLER_RESOURCE_REBALANCE_STRATEGY =
       AutoRebalanceStrategy.class.getName();
-  private static final int DEFAULT_LEAD_CONTROLLER_RESOURCE_REBALANCE_DELAY_MS 
= 300_000; // 5 minutes
-  private static final String DEFAULT_DIM_TABLE_MAX_SIZE = "200M";
-  private static final int UNSPECIFIED_THREAD_POOL = -1;
+  public static final int DEFAULT_LEAD_CONTROLLER_RESOURCE_REBALANCE_DELAY_MS 
= 300_000; // 5 minutes
+  public static final String DEFAULT_DIM_TABLE_MAX_SIZE = "200M";
+  public static final int UNSPECIFIED_THREAD_POOL = -1;
 
-  private static final String DEFAULT_PINOT_FS_FACTORY_CLASS_LOCAL = 
LocalPinotFS.class.getName();
+  public static final String DEFAULT_PINOT_FS_FACTORY_CLASS_LOCAL = 
LocalPinotFS.class.getName();
 
   public static final String DISABLE_GROOVY = 
"controller.disable.ingestion.groovy";
   public static final boolean DEFAULT_DISABLE_GROOVY = true;


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to