[GitHub] [flink] aljoscha opened a new pull request #10474: [FLINK-15116] Make JobClient stateless, remove AutoCloseable

[GitBox <gi...@apache.org>]

aljoscha opened a new pull request #10474: [FLINK-15116] Make JobClient stateless, remove AutoCloseable
URL: https://github.com/apache/flink/pull/10474
 
 
   ## What is the purpose of the change
   
   *Right now this is a proposal to show others what it looks like. If we agree on this changed behaviour I should add tests for it. Currently we don't have tests for `JobClient`, though.*
   
   With this change, the JobClient acquires the required ClusterClient for
   each method call. This means that the users no longer have the burden of
   managing the JobClient lifecycle, i.e. they can freely ignore the result
   of executeAsync().
   
   I also modified the `SocketWindowWordCount` to play around and show how the API works. **(This will be reverted)**.
   
   ## Brief change log
   
     - The first commit changes `ClusterDescriptor` to return a `ClusterClientProvider` so that consumers of the method can create a new `ClusterClient` whenever needed
     - The second commit changes `JobClient` to create a new `ClusterClient` from the provider for every method call. It therefore does not need to hold any resources that need to be cleaned up
   
   ## Verifying this change
   
   Same test coverage as before, we don't test the `JobClient` yet.
   
   ## Documentation
   
     - `JobClient` is not yet documented and this PR also doesn't add any documentation.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services