[FLINK-24562] YarnResourceManagerDriverTest should not use ContainerStatusPBImpl.newInstance

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.14.0
• Fix Version/s: 1.15.0
• Component/s: Deployment / YARN
• Labels: pull-request-available
• Assignee: Feifan Wang
• Reporter: Feifan Wang
• Created: 15/Oct/21 06:51
• Updated: 30/Nov/21 20:38
• Resolved: 20/Oct/21 02:21

Description

In YarnResourceManagerDriverTest, we create ContainerStatus with the static method ContainerStatusPBImpl{{.newInstance}}, which is annotated as private and unstable.

Although this method is still available in the latest version of yarn, some third-party versions of yarn may modify it. In fact, this method was modified in the internal version provided by our yarn team, which caused flink-1.14.0 to fail to compile.

Moreover, there is already an org.apache.flink.yarn.TestingContainerStatus, I think we should use it directly.
 

Issue Links

• links to: GitHub Pull Request #17496