tillrohrmann commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service) URL: https://github.com/apache/flink/pull/9643#issuecomment-529807926 Even though `JobExecutor` and `JobExecutorService` are not perfect it is clearly better than depending directly on the `MiniCluster` because it follows the [interface segregation principle](https://en.wikipedia.org/wiki/Interface_segregation_principle). As long as a user depends on `executeJobBlocking`, we should not remove it. Whether to use the client for communication with the `MiniCluster` or directly talking to it should not be topic of this PR. The same applies to the usage of the interface in the future. You might be right but at the moment we still have users of this interface and, hence, we should not do any second or third step before having done even the first. I still think that we should close this PR.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
