[YARN-7607] Remove the trailing duplicated timestamp in container diagnostics message

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.9.0
• Fix Version/s: 3.0.0, 2.10.0, 2.9.1
• Component/s: nodemanager
• Labels: log
• Target Version/s: 3.1.0, 2.9.1, 3.0.1
• Hadoop Flags: Reviewed
• Assignee: Weiwei Yang
• Reporter: Weiwei Yang
• Created: 05/Dec/17 05:12
• Updated: 13/Dec/17 08:41
• Resolved: 07/Dec/17 09:46

Description

There are some container diagnostic messages currently malformed, like below

###
017-12-05 11:43:21,319 INFO mapreduce.Job: map 28% reduce 0%
2017-12-05 11:43:22,345 INFO mapreduce.Job: Task Id : attempt_1512384455800_0003_m_000012_0, Status : FAILED
[2017-12-05 11:43:21.265]Container Killed to make room for Guaranteed Container[2017-12-05 11:43:21.265]
[2017-12-05 11:43:21.265]Container is killed before being launched.
###

such logs are presented both in console and RM UI, we need to remove the duplicated trailing timestamp from the log message. This is due to the mis-use of addDiagnostics function in these places.

Issue Links

• relates to: YARN-6306 NMClient API change for container upgrade