Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/6147#discussion_r195066482
--- 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 don't know that part of the code, my guess was that we want to keep the
transition from `Plan -> JobGraph` straight-forward.
---
