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

Posted to issues@arrow.apache.org by "Antoine Pitrou (JIRA)" <ji...@apache.org> on 2018/06/29 15:20:00 UTC

[jira] [Commented] (ARROW-2399) [Rust] Builder should not provide a set() method

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

Antoine Pitrou commented on ARROW-2399:
---------------------------------------

Chiming in again: while arrays are immutable, buffers need not be immutable. In C++, we have both mutable and immutable buffers (and both kinds are allowed to back arrays).

> [Rust] Builder<T> should not provide a set() method
> ---------------------------------------------------
>
>                 Key: ARROW-2399
>                 URL: https://issues.apache.org/jira/browse/ARROW-2399
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: Rust
>            Reporter: Andy Grove
>            Priority: Major
>             Fix For: 0.10.0
>
>
> Arrays should be immutable, but we have a `set` method on Buffer<T> that should not be there.
> This is only used from the Bitmap struct. Perhaps Bitmap should maintain its own memory instead and not use Buffer?



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