[SQOOP-1988] Sqoop2: isNull handling should be moved to CSVIntermediateDataFormat

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.99.5
• Assignee: Qian Xu
• Reporter: Qian Xu
• Created: 09/Jan/15 03:37
• Updated: 23/Jan/15 07:46
• Resolved: 23/Jan/15 06:51

Description

The Matcher.getMatchingData method is expected to rearrange record fields according to the FROM and TO schema. Currently here is an extra step in the implementation, which will reset any null "NULL" "null" "'null'" or "" field to null.

As there is no comment or documentation about this, I guess it is some undocumented special handling. Here is some discussion.

I think this check should not belong here. I propose to remove it. As the method will be called very frequently, the code removal will have performance advance. Thanks jerrychenhf

Issue Links

• links to: RB