GitHub user zentol opened a pull request: https://github.com/apache/flink/pull/6199
[FLINK-9624][runtime] Move jar/artifact upload out of jobgraph ## What is the purpose of the change This PR moves the logic for uploading jars/artifacts from the jobgraph into a separate utility class usable by all submission methods. The new `ClientUtils` class exposes 2 methods for uploading jars/artifacts and setting the respective blob keys on the `JobGraph`. All existing job-submission method were updated to use the new utilities and should now behave the same. The subsumed methods in `JobGraph` were removed, but remnants of them remain in 2 added methods: * setUserArtifactBlobKey sets the blobkey for a specific entry * finalizeUserArtifactEntries writes the artifact entries into the `ExecutionConfig` ## Verifying this change * `ClientUtils` is tested in `ClientUtilsTest` * `JobGraph` changes are covered in `JobGraphTest` * client modifications are covered by various existing tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/zentol/flink 9280_delta Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/6199.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #6199 ---- commit 13e3dc7dc9c0b7205223368a460993a309cb58ad Author: zentol <chesnay@...> Date: 2018-06-13T16:21:21Z [FLINK-9624][runtime] Move jar/artifact upload out of jobgraph ---- ---