You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "Jason Altekruse (JIRA)" <ji...@apache.org> on 2015/03/05 00:43:38 UTC
[jira] [Updated] (DRILL-2143) Remove RecordBatch from setup method
of DrillFunc interface
[ https://issues.apache.org/jira/browse/DRILL-2143?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jason Altekruse updated DRILL-2143:
-----------------------------------
Attachment: DRILL-2143-part1-mar-3.patch
DRILL-2143-part2-mar-3.patch
> 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-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)