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