[YARN-1120] Make ApplicationConstants.Environment.USER definition OS neutral

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.0.0-alpha1
• Fix Version/s: 2.1.1-beta
• Hadoop Flags: Reviewed
• Assignee: Chuan Liu
• Reporter: Chuan Liu
• Created: 28/Aug/13 23:01
• Updated: 12/May/16 18:29
• Resolved: 02/Sep/13 02:33

Description

In YARN-557, we added some code to make ApplicationConstants.Environment.USER has OS-specific definition in order to fix the unit test TestUnmanagedAMLauncher. In YARN-571, the relevant test code was corrected. In YARN-602, we actually will explicitly set the environment variables for the child containers. With these changes, I think we can revert the YARN-557 change to make ApplicationConstants.Environment.USER OS neutral. The main benefit is that we can use the same method over the Enum constants. This should also fix the TestContainerLaunch#testContainerEnvVariables failure on Windows.

Issue Links

• is part of: YARN-191 Enhancements to YARN for Windows Server and Windows Azure development and runtime environments