alamb commented on issue #10626:
URL: https://github.com/apache/datafusion/issues/10626#issuecomment-2126752102

   I was thinking about this last night -- what I would suggest for this is to 
   
   make functions like this for each type of 
https://docs.rs/parquet/latest/parquet/file/statistics/struct.ValueStatistics.html
 
   
   
   
   ```rust
   /// Returns an iterator over min values stored in `ValueStatistics<i32>`
   fn extract_i32_mins(stats: impl IntoIterator<&Statistics>) -> impl 
Iterator<Item = Option<i32>> {
   ...
   }
   ```
   
   And then with those iterators, we can make the arrays directly
   
   something like
   
   ```rust
   let Int32ArrayMins = Int32Aray::from_iter(extract_i32_mins(stats));
   ```
   
   I think that would be both simple and fast.


-- 
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.

To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to