git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[jira] [Created] (ARROW-2521) [Rust] Review Array design before first release


Andy Grove created ARROW-2521:
---------------------------------

             Summary: [Rust] Review Array design before first release
                 Key: ARROW-2521
                 URL: https://issues.apache.org/jira/browse/ARROW-2521
             Project: Apache Arrow
          Issue Type: Improvement
          Components: Rust
            Reporter: Andy Grove
            Assignee: Andy Grove
             Fix For: 0.10.0


Early on, [~kszucs] and I worked on two different designs for how to represent Arrow arrays in Rust, each with their pros and cons.

Krisztian started out with a generics approach e.g. Array<T> which was great until we tried to implement structs, which can contain mixed types so we ended up using enum to represent arrays, which was great until I got to the list types ... I don't think I can implement nested lists with this approach.

I am reviewing this again now that I am more familiar with Arrow and also my Rust skills have improved greatly since I started working on all of this.

I will be prototyping in a separate repo, and will update this Jira once I have something concrete to share, but I feel it is important to address this before the first official release of the Rust version. Also, if we are going to consider a refactor like this, it is better to do it now while the codebase is tiny.

 

 

 

 

 



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



( ! ) Warning: include(msgfooter.php): failed to open stream: No such file or directory in /var/www/git/apache-arrow-development/msg04311.html on line 106
Call Stack
#TimeMemoryFunctionLocation
10.0006364632{main}( ).../msg04311.html:0

( ! ) Warning: include(): Failed opening 'msgfooter.php' for inclusion (include_path='.:/var/www/git') in /var/www/git/apache-arrow-development/msg04311.html on line 106
Call Stack
#TimeMemoryFunctionLocation
10.0006364632{main}( ).../msg04311.html:0