rkhachatryan commented on PR #24869: URL: https://github.com/apache/flink/pull/24869#issuecomment-2154873780
> How about we simply add a set(ConfigOption) and set(String) to ExecutionConfig? It is not really necessary that ExecutionConfig picks up all configuration from ReadableConfig configuration. For a later runtime patching of ExecutionConfig, we could only use the set(String). We'd still need to call these methods with specific constants or keys, right? And doing so during run time would be problematic (the runtime would need to know these constants). -- 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
