paddyhoran opened a new pull request #7004:
URL: https://github.com/apache/arrow/pull/7004


   Replaces #6209 due to git issues.
   
   Implements UnionArray. This PR was getting too big as it was so I will 
address the following as follow up PR's:
   
   ARROW-8546
   ARROW-8547
   
   Note that I believe that we should split up array.rs and use separate 
underlying modules for each of the array types, keeping the Array trait and 
some common functionality in array.rs.
   
   Having a single file for all the array types worked fine to begin with but 
now it's getting bloated. This PR assumes we agree on this (at which point I'll 
open a follow up PR to move the other array types), if you disagree I can 
update this to move UnionArray back into array.rs, etc.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to