kl0u commented on a change in pull request #9690: [FLINK-14067] Decouple
PlanExecutor from JSON plan generation
URL: https://github.com/apache/flink/pull/9690#discussion_r325045322
##########
File path:
flink-java/src/main/java/org/apache/flink/api/java/ExecutionEnvironment.java
##########
@@ -766,13 +766,14 @@ public JobExecutionResult execute() throws Exception {
/**
* Creates the plan with which the system will execute the program, and
returns it as
* a String using a JSON representation of the execution data flow
graph.
- * Note that this needs to be called, before the plan is executed.
*
* @return The execution plan of the program, as a JSON String.
- * @throws Exception Thrown, if the compiler could not be instantiated,
or the master could not
- * be contacted to retrieve information relevant to
the execution planning.
+ * @throws Exception Thrown, if the compiler could not be instantiated.
*/
- public abstract String getExecutionPlan() throws Exception;
+ public String getExecutionPlan() throws Exception {
Review comment:
I think you can remove the `Exception`.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
