Awesome06 commented on issue #55418: URL: https://github.com/apache/arrow/issues/55418#issuecomment-6719510662
There appear to be two ways to resolve this: 1. Remove the Cython `list` type restriction on `RecordBatch.from_arrays(arrays, ...)` to accept any sequence, matching `Table.from_arrays` and the `Sequence[...]` typing stubs in #55264. 2. Keep `RecordBatch.from_arrays` restricted to lists and convert tuples via `list(data)` inside `pa.record_batch()`. Option 1 provides consistency across both tabular constructors. Does removing the `list` restriction on `RecordBatch.from_arrays` match the intended design? Happy to put together a PR with tests for (1) if that is the preferred direction. -- 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]
