kl0u commented on a change in pull request #13647: URL: https://github.com/apache/flink/pull/13647#discussion_r506177083
########## File path: flink-streaming-java/src/main/java/org/apache/flink/streaming/api/environment/StreamExecutionEnvironment.java ########## @@ -792,6 +792,12 @@ public void configure(ReadableConfig configuration, ClassLoader classLoader) { }); config.configure(configuration, classLoader); checkpointCfg.configure(configuration); + configuration.getOptional(ExecutionOptions.SORT_INPUTS).ifPresent( Review comment: The way it is now though, the user has to explicitly state that he/she wants the BATCH state backend and timer service, right? Not the other way around. I would expect that even when we have the options available, when the `runtime-mode` is decided to be BATCH, the default behaviour would be to use the batch statebackend and timerservice. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org