[MAPREDUCE-6818] Remove direct reference to TimelineClientImpl

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.9.0, 3.0.0-beta1, YARN-5355
• Labels: newbie++
• Target Version/s: YARN-5355
• Hadoop Flags: Reviewed
• Assignee: Li Lu
• Reporter: Li Lu
• Created: 06/Dec/16 21:34
• Updated: 21/Oct/17 06:28
• Resolved: 09/Dec/16 02:20

Description

sjlee0's quick audit shows that things that are referencing TimelineClientImpl directly today:

JobHistoryFileReplayMapperV1 (MR)
SimpleEntityWriterV1 (MR)
TestDistributedShell (DS)
TestDSAppMaster (DS)
TestNMTimelinePublisher (node manager)
TestTimelineWebServicesWithSSL (AHS)

This is not the right way to use TimelineClient and we should avoid direct reference to TimelineClientImpl as much as possible.

Any newcomers to the community are more than welcome to take this. If this remains unassigned for ~24hrs I'll jump in and do a quick fix.

Issue Links

• relates to: YARN-4675 Reorganize TimelineClient and TimelineClientImpl into separate classes for ATSv1.x and ATSv2