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`.
---
