pitrou commented on issue #35450: URL: https://github.com/apache/arrow/issues/35450#issuecomment-1647580124
Record batches are a bit different from struct arrays. Slicing a struct array changes the offset of the struct array itself. Slicing a record batch slices each column individually, since record batches do not have an offset. So there's no need to allow record batch columns with a different length than the record batch itself. > If we want to align on RecordBatch::Validate then I think we need to update RecordBatch::FromStructArray to check for this case and repair it (instead of using the struct array's length directly). That sounds like the best solution to me. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
