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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/12/03 23:09:49 UTC

[GitHub] [arrow] seen commented on pull request #8827: ARROW-10800 [Rust] [Parquet] Provide access to the elements of parquet::record::{List, Map}

seen commented on pull request #8827:
URL: https://github.com/apache/arrow/pull/8827#issuecomment-738434914


   My intention is to provide safe access to the contents of both `List` and `Map` and thought providing just a slice to those contents is sufficient. 
   
   Thinking more about it I wonder if instead of returning `&[Field]`, `List::elements` should instead return `Vec<&Field>`. I'm relatively new to rust so I'm not entirely sure the best API to provide for this information and worry that exposing just `&[Field]` provides an avenue for moving `Field` out from under `List`.


----------------------------------------------------------------
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:
users@infra.apache.org