[
https://issues.apache.org/jira/browse/FLINK-10304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16613564#comment-16613564
]
ASF GitHub Bot commented on FLINK-10304:
----------------------------------------
TisonKun commented on issue #6673: [FLINK-10304] [client] Remove deprecated
AbstractYarnClusterDescriptor field
URL: https://github.com/apache/flink/pull/6673#issuecomment-421026733
cc @GJL
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Remove deprecated AbstractYarnClusterDescriptor field
> -----------------------------------------------------
>
> Key: FLINK-10304
> URL: https://issues.apache.org/jira/browse/FLINK-10304
> Project: Flink
> Issue Type: Improvement
> Components: Client, YARN
> Affects Versions: 1.7.0
> Reporter: 陈梓立
> Assignee: 陈梓立
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.7.0
>
>
> Depend on [~trohrm...@apache.org]'s
> [commit|https://github.com/apache/flink/commit/6356128865bff7463bf03185d18b129ed3633bc2],
> {{AbstractYarnClusterDescriptor}} should not care whether it is in DETACHED
> mode.
> After digging I found the main usages of it are
> 1. {{FlinkYarnSessionCli#run}}, this can be resolved by checking whether
> {{allOptions}} has {{DETACHED_OPTION}} locally.
> 2. when AbstractYarnClusterDescriptor start a AM, it sets
> {{appMasterEnv.put(YarnConfigKeys.ENV_DETACHED, String.valueOf(detached));}}.
> At this point it seems that YarnClusterDescriptor should know whether or not
> it is in detached mode.
> If usage 2 is irrelevant now, we can get rid of deprecated method in FLIP-6
> codebase.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
