suddendust commented on a change in pull request #7173: URL: https://github.com/apache/incubator-pinot/pull/7173#discussion_r672488420
########## File path: pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java ########## @@ -428,15 +496,20 @@ public void setRetentionControllerFrequencyInSeconds(int retentionFrequencyInSec } /** - * Returns the config value for controller.offline.segment.interval.checker.frequencyInSeconds if it exists. - * If it doesn't exist, returns the segment level validation interval. This is done in order to retain the current behavior, - * wherein the offline validation tasks were done at segment level validation interval frequency - * The default value is the new DEFAULT_OFFLINE_SEGMENT_INTERVAL_CHECKER_FREQUENCY_IN_SECONDS + * Returns the config value for controller.offline.segment.interval.checker.frequencyInSeconds if Review comment: Missed this, will update the docs. -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org 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