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

Reply via email to