npawar commented on a change in pull request #5793:
URL: https://github.com/apache/incubator-pinot/pull/5793#discussion_r467323322



##########
File path: 
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/relocation/SegmentRelocator.java
##########
@@ -40,41 +41,51 @@
 
 
 /**
- * Periodic task to run rebalancer in background to relocate COMPLETED 
segments for LLC real-time table. Allow at most
- * one replica unavailable during rebalance.
+ * Periodic task to run rebalancer in background to
+ * 1. relocate COMPLETED segments to tag overrides
+ * 2. relocate ONLINE segments to tiers if tier configs are set
+ * Allow at most one replica unavailable during rebalance. Not applicable for 
HLC tables.
  */
-public class RealtimeSegmentRelocator extends ControllerPeriodicTask<Void> {
-  private static final Logger LOGGER = 
LoggerFactory.getLogger(RealtimeSegmentRelocator.class);
+public class SegmentRelocator extends ControllerPeriodicTask<Void> {
+  private static final Logger LOGGER = 
LoggerFactory.getLogger(SegmentRelocator.class);
 
   private final ExecutorService _executorService;
 
-  public RealtimeSegmentRelocator(PinotHelixResourceManager 
pinotHelixResourceManager,
+  public SegmentRelocator(PinotHelixResourceManager pinotHelixResourceManager,
       LeadControllerManager leadControllerManager, ControllerConf config, 
ControllerMetrics controllerMetrics,
       ExecutorService executorService) {
-    super("RealtimeSegmentRelocator", 
getRunFrequencySeconds(config.getRealtimeSegmentRelocatorFrequency()),
-        config.getRealtimeSegmentRelocationInitialDelayInSeconds(), 
pinotHelixResourceManager, leadControllerManager,
+    super(SegmentRelocator.class.getSimpleName(), 60,
+        30, pinotHelixResourceManager, leadControllerManager,

Review comment:
       removed!




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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

Reply via email to