[SPARK-2209] Cast shouldn't do null check twice

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0.0
• Fix Version/s: 1.0.1, 1.1.0
• Component/s: SQL
• Assignee: Reynold Xin
• Reporter: Reynold Xin
• Created: 20/Jun/14 02:44
• Updated: 20/Jun/14 07:01
• Resolved: 20/Jun/14 07:01

Description

Cast does two null checks, one in eval and another one in the function returned by nullOrCast. It's best to get rid of the one in nullOrCast (since eval will be the more common code path).