AlenkaF commented on a change in pull request #11457:
URL: https://github.com/apache/arrow/pull/11457#discussion_r731702512
##########
File path: python/pyarrow/table.pxi
##########
@@ -571,6 +571,10 @@ cdef _schema_from_arrays(arrays, names, metadata,
shared_ptr[CSchema]* schema):
if metadata is not None:
c_meta = KeyValueMetadata(metadata).unwrap()
+ if len(names) != K:
Review comment:
Thinking I shouldn't mess with reordering as it breaks things. Maybe I
would add a check like so?
```python
if K == 0:
if len(names) == 0:
schema.reset(new CSchema(c_fields, c_meta))
return arrays
else:
raise ValueError('Length of names ({}) does not match '
'length of arrays ({})'.format(len(names), K))
```
and leave the order as it was?
--
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]
