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:
[email protected]


Reply via email to