[YARN-4078] getPendingResourceRequestForAttempt is present in AbstractYarnScheduler should be present in YarnScheduler interface instead

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.8.0, 3.0.0-alpha1
• Component/s: resourcemanager
• Target Version/s: 2.8.0
• Hadoop Flags: Reviewed
• Assignee: Naganarasimha G R
• Reporter: Naganarasimha G R
• Created: 25/Aug/15 05:06
• Updated: 30/Aug/16 01:17
• Resolved: 16/Sep/15 07:01

Description

Currently getPendingResourceRequestForAttempt is present in AbstractYarnScheduler.
But in AppInfo, we are calling this method by typecasting it to AbstractYarnScheduler, which is incorrect.

Because if a custom scheduler is to be added, it will implement YarnScheduler, not AbstractYarnScheduler.

This method should be moved to YarnScheduler or it should have a guarded check like in other places (RMAppAttemptBlock.getBlackListedNodes)

Issue Links

• is related to: YARN-3986 getTransferredContainers in AbstractYarnScheduler should be present in YarnScheduler interface instead