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_r221169522
 
 

 ##########
 File path: 
flink-optimizer/src/main/java/org/apache/flink/optimizer/plan/OptimizedPlan.java
 ##########
 @@ -46,25 +46,20 @@
        /** The original program (as a dataflow plan). */
        private final Plan originalProgram;
 
-       /** Name of the job */
-       private final String jobName;
-
        /**
         * Creates a new instance of this optimizer plan container. The plan is 
given and fully
         * described by the data sources, sinks and the collection of all nodes.
-        * 
+        *
         * @param sources The data sources.
         * @param sinks The data sinks.
         * @param allNodes A collection containing all nodes in the plan.
-        * @param jobName The name of the program
         */
        public OptimizedPlan(Collection<SourcePlanNode> sources, 
Collection<SinkPlanNode> sinks,
-                       Collection<PlanNode> allNodes, String jobName, Plan 
programPlan)
+                                                Collection<PlanNode> allNodes, 
Plan programPlan)
 
 Review comment:
   Please revert the indentation.

----------------------------------------------------------------
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