[ 
https://issues.apache.org/jira/browse/FLINK-8342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16323041#comment-16323041
 ] 

ASF GitHub Bot commented on FLINK-8342:
---------------------------------------

Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5228#discussion_r161087011
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java
 ---
    @@ -99,7 +99,7 @@
     /**
      * The descriptor with deployment information for spawning or resuming a 
{@link YarnClusterClient}.
      */
    -public abstract class AbstractYarnClusterDescriptor implements 
ClusterDescriptor<YarnClusterClient> {
    +public abstract class AbstractYarnClusterDescriptor implements 
ClusterDescriptor {
        private static final Logger LOG = 
LoggerFactory.getLogger(YarnClusterDescriptor.class);
    --- End diff --
    
    Not intended. Will change it.


> Remove ClusterClient generic type parameter from ClusterDescriptor
> ------------------------------------------------------------------
>
>                 Key: FLINK-8342
>                 URL: https://issues.apache.org/jira/browse/FLINK-8342
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Client
>    Affects Versions: 1.5.0
>            Reporter: Till Rohrmann
>            Assignee: Till Rohrmann
>              Labels: flip-6
>             Fix For: 1.5.0
>
>
> The {{ClusterDescriptor}} should not specialize the returned 
> {{ClusterClient}} type in order to develop code which can work with all 
> {{ClusterDescriptors}} and {{ClusterClients}}. Therefore, I propose to remove 
> the generic type parameter from {{ClusterDescriptor}}.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to