[FLINK-10304] Remove deprecated AbstractYarnClusterDescriptor field

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Won't Do
• Affects Version/s: 1.7.0
• Component/s: Command Line Client, Deployment / YARN
• Labels: pull-request-available
• Assignee: Zili Chen
• Reporter: Zili Chen
• Created: 07/Sep/18 16:16
• Updated: 17/Jan/19 19:22
• Resolved: 17/Jan/19 19:22

Description

Depend on [e-mail]'s commit, 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.

Issue Links

• links to: GitHub Pull Request #6673