[YARN-4775] Nodemanager checks for old aggregated logs as the NM user instead of the app user

• Type: Bug
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 2.6.0
• Component/s: nodemanager
• Target Version/s: 2.7.5
• Assignee: Unassigned
• Reporter: Jason Darrell Lowe
• Created: 08/Mar/16 15:42
• Updated: 28/Jul/17 00:54

Description

AppLogAggregatorImpl#cleanOldLogs checks for old aggregated logs as the NM user instead of using the UGI for the app whose logs are being aggregated. This causes an extra connection to the namenode if the NM user wasn't already connected to the namenode. If cleanOldLogs does find logs to delete then it uses the UGI of the app to delete them, so it's not consistent. It should use the app UGI for both listing and for deleting.