aljoscha commented on a change in pull request #8931: [FLINK-13041] Make 
ScheduleMode configurable via ExecutionConfig
URL: https://github.com/apache/flink/pull/8931#discussion_r299368009
 
 

 ##########
 File path: 
flink-core/src/main/java/org/apache/flink/api/common/ExecutionConfig.java
 ##########
 @@ -90,6 +90,8 @@
        /** Defines how data exchange happens - batch or pipelined */
        private ExecutionMode executionMode = ExecutionMode.PIPELINED;
 
+       private ScheduleMode scheduleMode = ScheduleMode.LAZY_FROM_SOURCES;
 
 Review comment:
   Ha! you got me! I left this in on purpose because I wanted to see if all the 
tests still pass with this different default. We should of course not have this 
in the final code.
   
   I put it in `ExecutionConfig` because it is available in `flink-core` while 
`StreamExecutionEnvironment` is not. I had a feeling that maybe the Table API 
will need access to this at some point or other. What do you think?

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to