[jira] [Created] (IMPALA-5720) FunctionContext::*Allocate*() should use TryAllocate() instead of Allocate().

["Tim Armstrong (JIRA)" <ji...@apache.org>]

Tim Armstrong created IMPALA-5720:
-------------------------------------

             Summary: FunctionContext::*Allocate*() should use TryAllocate() instead of Allocate().
                 Key: IMPALA-5720
                 URL: https://issues.apache.org/jira/browse/IMPALA-5720
             Project: IMPALA
          Issue Type: Improvement
          Components: Backend
    Affects Versions: Impala 2.10.0
            Reporter: Tim Armstrong
            Assignee: Tim Armstrong


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.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)