[TEZ-3337] Do not log empty fields of TaskAttemptFinishedEvent to avoid confusion

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.9.0, 0.8.5
• Assignee: Zhiyuan Yang
• Reporter: Zhiyuan Yang
• Created: 12/Jul/16 21:43
• Updated: 14/Mar/17 03:49
• Resolved: 13/Jul/16 17:55

Description

For successful task attempt, we don't record the containerId, which cause "containerId=," in the INFO logs. We should avoid logging this field if it's empty.

2016-07-07 22:49:44,935 [INFO] [Dispatcher thread {Central}] |history.HistoryEventHandler|: [HISTORY][DAG:dag_1467855578616_0044_1][Event:TASK_ATTEMPT_FINISHED]: vertexName=Map 1, taskAttemptId=attempt_1467855578616_0044_1_07_000000_0, creationTime=1467956979891, allocationTime=1467956980426, startTime=1467956982433, finishTime=1467956984933, timeTaken=2500, status=SUCCEEDED, errorEnum=, diagnostics=, containerId=, nodeId=, nodeHttpAddress=
2016-07-07 22:49:44,937 [INFO] [Dispatcher thread {Central}] |history.HistoryEventHandler|: [HISTORY][DAG:dag_1467855578616_0044_1][Event:TASK_ATTEMPT_FINISHED]: vertexName=Map 11, taskAttemptId=attempt_1467855578616_0044_1_02_000000_0, creationTime=1467956979894, allocationTime=1467956980427, startTime=1467956982437, finishTime=1467956984936, timeTaken=2499, status=SUCCEEDED, errorEnum=, diagnostics=, containerId=, nodeId=, nodeHttpAddress=