You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "TisonKun (JIRA)" <ji...@apache.org> on 2018/12/13 03:57:00 UTC
[jira] [Updated] (FLINK-11146) Get rid of legacy codes from
ClusterClient
[ https://issues.apache.org/jira/browse/FLINK-11146?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
TisonKun updated FLINK-11146:
-----------------------------
Issue Type: Task (was: Sub-task)
Parent: (was: FLINK-10392)
> Get rid of legacy codes from ClusterClient
> ------------------------------------------
>
> Key: FLINK-11146
> URL: https://issues.apache.org/jira/browse/FLINK-11146
> Project: Flink
> Issue Type: Task
> Components: Client, Tests
> Affects Versions: 1.8.0
> Reporter: TisonKun
> Assignee: TisonKun
> Priority: Major
> Fix For: 1.8.0
>
>
> As [~StephanEwen] mentioned in ML, the client needs big refactoring / cleanup. It should use a proper HTTP client library to help with future authentication mechanisms.
> After an investigation I notice that the valid cluster clients are only {{MiniClusterClient}} and {{RestClusterClient}}. Legacy clients, {{StandaloneClusterClient}} and {{YarnClusterClient}}, as well as pre-FLIP-6 codes inside {{ClusterClient}}, should be removed as part of FLINK-10392. With this removal we arrive a clean stage where we can think how to implement a proper HTTP client more comfortably.
> 1. {{StandaloneClusterClient}} is now depended on by {{LegacyStandaloneClusterDescriptor}} (the removal is tracked by FLINK-10700) and {{FlinkClient}}(part of flink-storm which is decided to be removed FLINK-10571). Also relevant tests need to be ported(or directly removed).
> 2. The removal of {{YarnClusterClient}} should go along with FLINK-11106 Remove legacy flink-yarn component.
> 3. Testing classes inheriting from {{ClusterClient}} need to be ported(or directly removed).
> 4. Get rid of legacy codes inside {{ClusterClient}} it self, such as {{#run(JobGraph, ClassLoader)}}
> Besides, what is {{JobClient}} used for? I cannot find valid usages of it.
> cc [~mxm] [~till.rohrmann]
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)