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

Jeroen van Straten commented on ARROW-7051:
-------------------------------------------

I wouldn't call this for every buffer; I would do it in conjunction with the 
power-of-two size buffer size caching that you suggested in the first comment. 
But whether it's worth doing (and worth the additional maintenance), I don't 
know.

> [C++] Improve MakeArrayOfNull to support creation of multiple arrays
> --------------------------------------------------------------------
>
>                 Key: ARROW-7051
>                 URL: https://issues.apache.org/jira/browse/ARROW-7051
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>    Affects Versions: 0.14.0
>            Reporter: Ben Kietzman
>            Assignee: Jeroen van Straten
>            Priority: Minor
>              Labels: beginner, good-first-issue
>          Time Spent: 1h 40m
>  Remaining Estimate: 0h
>
> MakeArrayOfNull reuses a single buffer of {{0}} for all buffers in the array 
> it creates. It could be extended to reuse that same buffer for all buffers in 
> multiple arrays. This optimization will make RecordBatchProjector and 
> ConcatenateTablesWithPromotion more memory efficient



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to