[jira] [Resolved] (DRILL-2143) Remove RecordBatch from setup method of DrillFunc interface

["Jason Altekruse (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/DRILL-2143?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jason Altekruse resolved DRILL-2143.
------------------------------------
    Resolution: Fixed

Resolved in bff7b9ef5a9f345908aca160a97b98f6ab187708 and 1c5decc17cf38cbf4a4119d7ca19653cb19e1b53

> Remove RecordBatch from setup method of DrillFunc interface
> -----------------------------------------------------------
>
>                 Key: DRILL-2143
>                 URL: https://issues.apache.org/jira/browse/DRILL-2143
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Functions - Drill
>            Reporter: Jason Altekruse
>            Assignee: Jason Altekruse
>             Fix For: 0.8.0
>
>         Attachments: DRILL-2143-part1-feb-27.patch, DRILL-2143-part1-feb-6.patch, DRILL-2143-part1-mar-3.patch, DRILL-2143-part2-15-mar-15.patch, DRILL-2143-part2-feb-27.patch, DRILL-2143-part2-feb-6.patch, DRILL-2143-part2-mar-3.patch, DRILL-2143-remove-record-batch-from-udfs.patch
>
>
> 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/%3CCAMpYv7AC_-9u4iRZ+5fxoenzBOJCTovJronn0qri4BQZf535Qw@mail.gmail.com%3E 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)