[SQOOP-1786] Sqoop2: Stop using JSONValue.parse method call

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.99.4
• Fix Version/s: 1.99.5
• Assignee: Jarek Jarcec Cecho
• Reporter: Jarek Jarcec Cecho
• Created: 23/Nov/14 05:07
• Updated: 26/Nov/14 11:55
• Resolved: 26/Nov/14 11:10

Description

While investigating SQOOP-1784, I've noticed that JSONValue.parse() method that we're using everywhere in the code base is swallowing exceptions and returning null on all failure scenarios.

As this is causing a random NullPointerException being thrown elsewhere without proper information what went wrong, I would like to migrate to method JSONValue.parseWithException() instead. Given how much we're parsing JSONs inside Sqoop 2, I would also recommend to create a small Util class that will handle error cases and coverts the ParserException to SqoopException.