dimakuz opened a new pull request, #809:
URL: https://github.com/apache/arrow-go/pull/809

   ### Rationale for this change
   
   IPC deserialization of `MapType` reconstructed map key/value children from 
only their data types, which dropped field-level metadata on the key and value 
fields. This could silently lose metadata such as field IDs after an IPC schema 
round-trip.
   
   ### What changes are included in this PR?
   
   The IPC metadata reader now reconstructs `MapType` using 
`arrow.MapOfFields(...)` with the already-deserialized key and value fields, 
preserving their metadata and nullability.
   
   A regression test was added to verify that map key/value field metadata, 
item nullability, and `KeysSorted` survive an IPC schema round-trip.
   
   ### Are these changes tested?
   
   Yes - unit test included.
   
   ### Are there any user-facing changes?
   
   Yes - IPC schema deserialization now preserves field-level metadata on 
MapType key and value children.


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