Myasuka commented on code in PR #23347: URL: https://github.com/apache/flink/pull/23347#discussion_r1360198299
########## flink-streaming-java/src/main/java/org/apache/flink/streaming/api/environment/ExecutionCheckpointingOptions.java: ########## @@ -314,4 +317,28 @@ public class ExecutionCheckpointingOptions { "Defines the maximum number of subtasks that share the same channel state file. " + "It can reduce the number of small files when enable unaligned checkpoint. " + "Each subtask will create a new channel state file when this is configured to 1."); + + public static final ConfigOption<ThreadDumpLogLevel> + THREAD_DUMP_LOG_LEVEL_WHEN_CHECKPOINT_TIMEOUT = + key("execution.checkpointing.thread-dump-log-level-when-checkpoint-timeout") Review Comment: First of all, I think not all scenarios obey the `execution.checkpointing` prefix. And I think making different log level for different scenarios would be a bit too complex. We only need two parameters no matter how many scenarios supported in my proposal. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org