Uploaded image for project: 'Apache Drill'
  1. Apache Drill
  2. DRILL-2143

Remove RecordBatch from setup method of DrillFunc interface

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.8.0
    • Component/s: Functions - Drill
    • Labels:
      None

      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/%3CCAMpYv7AC_-9u4iRZ+5fxoenzBOJCTovJronn0qri4BQZf535Qw@mail.gmail.com%3E

        Attachments

        1. DRILL-2143-part1-feb-27.patch
          183 kB
          Jason Altekruse
        2. DRILL-2143-part1-feb-6.patch
          204 kB
          Jason Altekruse
        3. DRILL-2143-part1-mar-3.patch
          184 kB
          Jason Altekruse
        4. DRILL-2143-part2-15-mar-15.patch
          40 kB
          Jason Altekruse
        5. DRILL-2143-part2-feb-27.patch
          39 kB
          Jason Altekruse
        6. DRILL-2143-part2-feb-6.patch
          39 kB
          Jason Altekruse
        7. DRILL-2143-part2-mar-3.patch
          39 kB
          Jason Altekruse
        8. DRILL-2143-remove-record-batch-from-udfs.patch
          232 kB
          Jason Altekruse

          Issue Links

            Activity

              People

              • Assignee:
                cchang@maprtech.com Chun Chang
                Reporter:
                jaltekruse Jason Altekruse
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: