[GitHub] [arrow] jorisvandenbossche commented on pull request #34691: GH-34639: [C++] Support RecordBatch::FromStructArray even if struct array has nulls/offsets

["jorisvandenbossche (via GitHub)" <gi...@apache.org>]

jorisvandenbossche commented on PR #34691:
URL: https://github.com/apache/arrow/pull/34691#issuecomment-1481305606

   Sounds good to me. 
   I don't know if there is a use case for wanting it to be zero-copy (like in the conversion to numpy there is a zero_copy_only keyword that avoids doing the conversion if it would trigger a copy). But in the end, the user can quite easily check the null_count of the parent struct array if they for some reason would want to raise an error in that case. 


-- 
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-unsubscribe@arrow.apache.org

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