Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-2756 Rethink UDF interfaces
  3. IMPALA-5720

FunctionContext::*Allocate*() should use TryAllocate() instead of Allocate().

    XMLWordPrintableJSON

    Details

      Description

      Currently the UDF interface uses Allocate(), which only returns NULL if malloc() fails. We should switch it to use TryAllocate() instead so that the query stays within the memory limit.

      We should consider this a breaking change because existing UDFs and UDAs may not tolerate Allocate() returning NULL very well, and this would make it more frequent.

      I would like to get this in soon, but put it behind a flag so that it can optionally be enabled.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                tarmstrong Tim Armstrong
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: