[SPARK-8993] More comprehensive type checking in expressions

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.5.0
• Component/s: SQL
• Target Version/s: 1.5.0
• Assignee: Reynold Xin
• Reporter: Reynold Xin
• Created: 10/Jul/15 23:06
• Updated: 15/Jul/15 05:52
• Resolved: 15/Jul/15 05:52

Description

This patch makes the following changes:

1. ExpectsInputTypes only defines expected input types, but does not perform any implicit type casting.
2. ImplicitCastInputTypes is a new trait that defines both expected input types, as well as performs implicit type casting.
3. BinaryOperator has a new abstract function "inputType", which defines the expected input type for both left/right. Concrete BinaryOperator expressions no longer perform any implicit type casting.
4. For BinaryOperators, convert NullType (i.e. null literals) into some accepted type so BinaryOperators don't need to handle NullTypes.

Issue Links

• is duplicated by: SPARK-8969 move type-check from BinaryArithmetic and BinaryComparison to BinaryOperator
• links to: [Github] Pull Request #7348 (rxin)