Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/6147#discussion_r195044146 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/jobgraph/JobGraph.java --- @@ -581,6 +582,8 @@ public String toString() { } public void uploadUserArtifacts(InetSocketAddress blobServerAddress, Configuration clientConfig) throws IOException { + zipUserArtifacts(); --- End diff -- Could we say that all `userArtifacts` are already zipped if they are a directory before being added via `addUserArtifact`. This means that the caller is responsible for the zipping. That way we could get rid of modifying the `JobGraph` as a side effect of `uploadUserArtifacts`.
---