[YARN-8640] Restore previous state in container-executor after failure

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.10.0, 3.2.0, 2.9.2, 2.8.5, 2.7.8, 3.0.4, 3.1.2
• Hadoop Flags: Reviewed
• Assignee: Jim Brennan
• Reporter: Jim Brennan
• Created: 09/Aug/18 14:59
• Updated: 17/Aug/18 20:22
• Resolved: 17/Aug/18 20:22

Description

The container-executor function write_exit_code_file_as_nm had a number of failure conditions where it just returns -1 without restoring previous state.
This is not a problem in any of the places where it is currently called, but it could be a problem if future code changes call it before code that depends on the previous state.