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_r219476983
##########
File path:
flink-optimizer/src/main/java/org/apache/flink/optimizer/plan/OptimizedPlan.java
##########
@@ -49,23 +49,28 @@
/** Name of the job */
private final String jobName;
+ /** Descrption of the job */
+ private final String jobDescription;
+
/**
* 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
+ * @param jobDescription The description of the program
*/
public OptimizedPlan(Collection<SourcePlanNode> sources,
Collection<SinkPlanNode> sinks,
- Collection<PlanNode> allNodes, String jobName, Plan
programPlan)
+ Collection<PlanNode> allNodes,
String jobName, Plan programPlan, String jobDescription)
Review comment:
Actually I would remove the `jobName` and `jobDescription` parameters, and
use those values from the programPlan, which we always do anyways.
----------------------------------------------------------------
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
