[YARN-7476] Fix miscellaneous issues in ATSv2 after merge to branch-2

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.9.0
• Hadoop Flags: Reviewed
• Assignee: Varun Saxena
• Reporter: Varun Saxena
• Created: 11/Nov/17 21:54
• Updated: 12/Nov/17 18:03
• Resolved: 12/Nov/17 17:57

Description

a) We are still using Resource#getMemory in NMTimelinePublisher#publishContainerCreatedEvent. This has been deprecated since YARN-4844. Better to use getMemorySize instead.
b) Post YARN-5865, application priority should be fetched from RMAppImpl instead of app submission context. But we are still fetching it from submission context while publishing entities to timeline service. This would mean that if priority is updated, it will not be published to timeline service.
c) The order of app_collectors in NodeHeartbeatResponseProto is different from trunk. Better to make it consistent.