[
https://issues.apache.org/jira/browse/FLINK-14915?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16979843#comment-16979843
]
Zili Chen commented on FLINK-14915:
-----------------------------------
Yes I think {{JobID}} and {{SavepointSettings}} is necessary for now. It
doesn't require the whole job graph which implies we need the topologies.
Would you like to send a minor patch on this issue?
> SchedulingStrategyFactory#createInstance might not need to know JobGraph
> ------------------------------------------------------------------------
>
> Key: FLINK-14915
> URL: https://issues.apache.org/jira/browse/FLINK-14915
> Project: Flink
> Issue Type: Improvement
> Components: Runtime / Coordination
> Reporter: Zili Chen
> Priority: Major
>
> [~zhuzh] [~GJL] I just notice that
> {{SchedulingStrategyFactory#createInstance}} take a parameter {{JobGraph}}
> but neither the parameter is in use nor this class/method should know
> {{JobGraph}}. Could you explain why we need it or we can safely remove the
> parameter so that we get rid of confusing parameter?
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
