xiangfu0 commented on code in PR #14991: URL: https://github.com/apache/pinot/pull/14991#discussion_r1942060957
########## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixSegmentOnlineOfflineStateModelGenerator.java: ########## @@ -46,6 +40,10 @@ private PinotHelixSegmentOnlineOfflineStateModelGenerator() { public static final String DROPPED_STATE = "DROPPED"; public static final String CONSUMING_STATE = "CONSUMING"; + public static final int DEFAULT_TRANSITION_PRIORITY = 1000; + // Given a lower priority to let the consuming segment start last during server startup. + public static final int OFFLINE_TO_CONSUMING_TRANSITION_PRIORITY = 1100; Review Comment: just give enough buffer between each state. -- 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