[ 
https://issues.apache.org/jira/browse/FLINK-10575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16787698#comment-16787698
 ] 

TisonKun commented on FLINK-10575:
----------------------------------

[~till.rohrmann] Besides remove the deprecated method, I notice that parameter 
{{prior}} is always {{null}}. It is still valid concept that we might attach a 
job graph to an existing execution graph?


{code:java}
       /**
         * Builds the ExecutionGraph from the JobGraph.
         * If a prior execution graph exists, the JobGraph will be attached. If 
no prior execution
         * graph exists, then the JobGraph will become attach to a new empty 
execution graph.
         */
        public static ExecutionGraph buildGraph(
                        @Nullable ExecutionGraph prior,
{code}


> Remove deprecated ExecutionGraphBuilder.buildGraph method
> ---------------------------------------------------------
>
>                 Key: FLINK-10575
>                 URL: https://issues.apache.org/jira/browse/FLINK-10575
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Runtime / Coordination
>    Affects Versions: 1.7.0
>            Reporter: JIN SUN
>            Assignee: JIN SUN
>            Priority: Major
>
> ExecutionGraphBuilder is not a public API and we should able to remove 
> deprecated method such as:
> @Deprecated
> public static ExecutionGraph buildGraph
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to