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

Posted to github@arrow.apache.org by "benibus (via GitHub)" <gi...@apache.org> on 2023/06/23 19:51:01 UTC

[GitHub] [arrow] benibus commented on issue #35450: [C++] RecordBatch::ToStructArray does not respect the length of the record batch if it differs from the length of the child arrays

benibus commented on issue #35450:
URL: https://github.com/apache/arrow/issues/35450#issuecomment-1604818908

   The docs for `Make` say that the columns' lengths should match the requested length, so I suppose it's technically illegal. https://github.com/apache/arrow/blob/ed99693319036f0b8fb6585858541cec7a1268ae/cpp/src/arrow/record_batch.h#L44-L50
   
   Whether that _should_ be the case is debatable though. If we were to slice the arrays then we'd probably want to do it on construction so methods like `Equals` would also work.


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