You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@arrow.apache.org by "Wes McKinney (JIRA)" <ji...@apache.org> on 2018/12/27 05:13:00 UTC

[jira] [Commented] (ARROW-4121) [C++] Remove memory allocation from InvertKernel

    [ https://issues.apache.org/jira/browse/ARROW-4121?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16729330#comment-16729330 ] 

Wes McKinney commented on ARROW-4121:
-------------------------------------

To be clear, we need to have a usable code path for both the case of 

* allocate memory and invoke kernel
* invoke kernel on pre-allocated memory

Right now we only have the first option. 

There's some code that partly handles this issue in kernels/cast.cc, but it could also benefit from some refactoring (to make the "allocate output space" problem more generic / reusable for other kernel implementations)

> [C++] Remove memory allocation from InvertKernel
> ------------------------------------------------
>
>                 Key: ARROW-4121
>                 URL: https://issues.apache.org/jira/browse/ARROW-4121
>             Project: Apache Arrow
>          Issue Type: Sub-task
>          Components: C++
>            Reporter: Micah Kornfield
>            Assignee: Micah Kornfield
>            Priority: Major
>
> subtask for overall fix to remove memory allocation.  Do this first to make sure I understand the problem and acceptable solution (seems like a simple API change).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)