[
https://issues.apache.org/jira/browse/FLINK-2483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14653579#comment-14653579
]
ASF GitHub Bot commented on FLINK-2483:
---------------------------------------
Github user zentol commented on the pull request:
https://github.com/apache/flink/pull/984#issuecomment-127595762
this change is not necessary imo. both in ExecutionGraph and JobGraph,
scheduleMode is initialized to a proper ScheduleMode. setScheduleMode requires
a ScheduleMode as an argument, so there's no issue here aswell. only possible
problem i can see is passing null to setScheduleMode, but that's catched in the
switch statement anyway.
> Add default branch of switch(scheduleMode) in scheduleForExecution function
> ---------------------------------------------------------------------------
>
> Key: FLINK-2483
> URL: https://issues.apache.org/jira/browse/FLINK-2483
> Project: Flink
> Issue Type: Bug
> Components: JobManager
> Affects Versions: 0.8.1
> Reporter: fangfengbin
> Priority: Minor
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
