[GitHub] [flink] TisonKun commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service)

2019-09-10 Thread GitBox
TisonKun commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service) URL: https://github.com/apache/flink/pull/9643#issuecomment-529840825 Now I agree with that we don't have to rush into the removal since there are still users of this interface. I will

[GitHub] [flink] TisonKun commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service)

2019-09-09 Thread GitBox
TisonKun commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service) URL: https://github.com/apache/flink/pull/9643#issuecomment-529727140 @flinkbot run travis This is an automated message

[GitHub] [flink] TisonKun commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service)

2019-09-09 Thread GitBox
TisonKun commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service) URL: https://github.com/apache/flink/pull/9643#issuecomment-529726901 Hi @tillrohrmann, thanks for your review. I don't think this is an issue with high priority, but let me share you

[GitHub] [flink] TisonKun commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service)

2019-09-09 Thread GitBox
TisonKun commented on issue #9643: [FLINK-13961][client] Remove obsolete abstraction JobExecutor(Service) URL: https://github.com/apache/flink/pull/9643#issuecomment-529364151 travis fails on NOTICE-binary checker. I don't think it is relevant to this pull request. Instead, it looks