ryzhyk opened a new pull request, #24790:
URL: https://github.com/apache/datafusion/pull/24790

   ## Which issue does this PR close?
   
   - Closes #24786.
   
   ## Rationale for this change
   
   `Int96Coercer` rebuilds every struct, list and map field with 
`Field::new_struct`, `Field::new_list` and `Field::new`, which start from empty 
metadata. Leaf fields go through `field_with_new_type`, which clones the field, 
so only container fields are affected, and only when the file holds an INT96 
column: the same file read without `coerce_int96` keeps its metadata.
   
   
   ## What changes are included in this PR?
   
   Carry `current_field.metadata()` across in the three constructors.
   
   ## Are these changes tested?
   
   yes
   
   ## Are there any user-facing changes?
   
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to