Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/5509#discussion_r168922479
--- Diff:
flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontend.java ---
@@ -1126,7 +1126,7 @@ public static void
setJobManagerAddressInConfig(Configuration config, InetSocket
* @return JobGraph extracted from the PackagedProgram
* @throws ProgramInvocationException if the JobGraph generation failed
*/
- private static JobGraph createJobGraph(Configuration configuration,
PackagedProgram packagedProgram, int defaultParallelism) throws
ProgramInvocationException {
+ public static JobGraph createJobGraph(Configuration configuration,
PackagedProgram packagedProgram, int defaultParallelism) throws
ProgramInvocationException {
--- End diff --
You're right, we should move it to a utility class.
---
