[MAPREDUCE-4878] JobID.forName() isn't strict enough

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Later
• Target Version/s: 2.1.0-beta, 1.3.0
• Assignee: Robert Kanter
• Reporter: Robert Kanter
• Created: 19/Nov/12 21:08
• Updated: 21/Mar/14 18:29
• Resolved: 21/Mar/14 18:29

Description

If you have a job running as job_201208221603_0003, and then try to kill a job passing id job_201208221603_003, it will kill job_201208221603_0003 because the last part of the JobID is parsed as an integer. We should make JobID.forName() stricter to prevent this and similar situations as the current behavior isn't so obvious.

More specifically, we shouldn't accept JobIDs if the last part of the JobID is:
- less than 4 characters (e.g. _003, _123)
- more than 4 characters and has a leading zero (e.g. _00003, _01234)