comphead commented on PR #5186:
URL: https://github.com/apache/arrow-rs/pull/5186#issuecomment-1992062531

   > Apologies for the delay, coming back to this and honestly a little 
confused as to the use-case. The updated error message uses the new field_names 
method, but the RecordBatch method itself is not concerned with nested fields, 
and so printing the nested fields is actually just confusing? It should only 
print the first level of field names, as that is all RecordBatch is concerned 
with?
   
   Thanks @tustvold for having the time for this. The PR is planned to make 
debugging easier and improve the error text which is currently too unclear.
    ```
   Arrow error: Invalid argument error: number of columns(2) must match number 
of fields(1) in schema
   ```
   
   The idea is to show which fields exists, and which are missing in the 
schema. But if the field is nested we should display it in user-friendly way. 
You are correct about using dots to separate parent field and child fields, 
this is not very good idea as dots also used in catalog/schema/table qualifiers.


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

Reply via email to