9aman commented on code in PR #15008: URL: https://github.com/apache/pinot/pull/15008#discussion_r1955604505
########## 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: Although for this to happen, all the servers should meet the disk utilization criteria at the same time. The segment build and upload should succeed even if one of the server's has disk left. -- 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