Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/6147#discussion_r195080145
--- 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 --
I think it would be better to change these parts instead of pushing this
logic into the `JobGraph`.
---
