dawidwys commented on a change in pull request #6266: [FLINK-9682] Add 
setDescription to execution environment and provide description field for the 
rest api
URL: https://github.com/apache/flink/pull/6266#discussion_r221169745
 
 

 ##########
 File path: 
flink-optimizer/src/main/java/org/apache/flink/optimizer/traversals/PlanFinalizer.java
 ##########
 @@ -76,7 +76,7 @@ public PlanFinalizer() {
                this.stackOfIterationNodes = new 
ArrayDeque<IterationPlanNode>();
        }
 
-       public OptimizedPlan createFinalPlan(List<SinkPlanNode> sinks, String 
jobName, Plan originalPlan) {
+       public OptimizedPlan createFinalPlan(List<SinkPlanNode> sinks, String 
jobName, Plan originalPlan, String jobDescription) {
 
 Review comment:
   `jobName` and `jobDescription` are not needed anymore.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to