[DRILL-2143] Remove RecordBatch from setup method of DrillFunc interface

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.8.0
• Component/s: Functions - Drill
• Assignee: Chun Chang
• Reporter: Jason Altekruse
• Created: 02/Feb/15 22:13
• Updated: 28/Apr/15 21:59
• Resolved: 17/Mar/15 22:20

Description

Drill UDFs currently are exposed to too much system state by receiving a reference to a RecordBatch in their setup method. This is not necessary as all of the schema change triggered operator functionality is handled outside of UDFs (the UDFS themselves are actually required to define a specific type they take as input, except in the case of complex types (maps and lists)). The only remaining artifact left from this interface is the date/time functions that ask for the query start time or current timezone. This can be provided to functions using a new injectable type, as DrillBufs are provided to functions currently. For more info read here: http://mail-archives.apache.org/mod_mbox/drill-dev/201501.mbox/[e-mail]%3E

Issue Links

• is depended upon by: DRILL-2060 Support constant folding in expressions in Filter
• is depended upon by: DRILL-2173 Enable querying partition information without reading all data