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