9aman commented on code in PR #15008: URL: https://github.com/apache/pinot/pull/15008#discussion_r1955592860
########## pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java: ########## @@ -320,6 +328,12 @@ private static long getRandomInitialDelayInSeconds() { 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_DISK_UTILIZATION_THRESHOLD = 0.99; Review Comment: I understand that the value is kept high to prevent false alerts but this can come at a cost of build failures for already ingested segments. Index creation, mmap etc might fail for these segments. We might need to reset these segments to bring them back to CONSUMING. For pausless, the DR will need to fix these. cc: @swaminathanmanish -- 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