[DRILL-3761] CastIntDecimal implementation should not update the input holder.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.2.0
• Component/s: Functions - Drill
• Assignee: Mehant Baid
• Reporter: Jinfeng Ni
• Created: 10/Sep/15 22:09
• Updated: 14/Jan/16 18:29
• Resolved: 14/Jan/16 18:29

Description

CastIntDecimal implementation would update the input holder's value, which may cause some side effect. This is especially true, when the run-time generated code tries to re-use the holder for common expressions.

In general, Drill's build-in/UDF implementation had better not modify the input holder.