[jira] [Created] (FLINK-14130) Remove ClusterClient.run() methods

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (FLINK-14130) Remove ClusterClient.run() methods

Shang Yuanchun (Jira)
Aljoscha Krettek created FLINK-14130:
----------------------------------------

             Summary: Remove ClusterClient.run() methods
                 Key: FLINK-14130
                 URL: https://issues.apache.org/jira/browse/FLINK-14130
             Project: Flink
          Issue Type: Sub-task
          Components: Client / Job Submission
            Reporter: Aljoscha Krettek


{{ClusterClient}} is anĀ internal interface of the {{flink-clients}} package. It should only be concerned with submitting {{JobGraphs}} to a cluster, which is what {{submitJob()}} does.

The {{run()}} methods are concerned with unpacking programs or job-with-jars and at the end use {{submitJob()}} in some way, they should reside in some other component. The only valid remaining run method is {{run(PackagedProgram prog, int parallelism)}}, this could be in {{PackagedProgramUtils}}. The other {{run()}} methods are actually only used in one test: {{ClientTest.shouldSubmitToJobClient()}}. I don't think that test is valid anymore, it evolved for a very long time and now doesn't test what it was supposed to test once.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)