jorgecarleitao commented on issue #1091:
URL: https://github.com/apache/arrow-rs/issues/1091#issuecomment-1000249937


   Oh, sorry, I skipped the 3rd item by mistake. I do not remember the reason. 
Had a quick look at the spec:
   
   * IPC format: each metadata is an array of key pairs 
([here](https://github.com/apache/arrow/blob/master/format/Schema.fbs#L481) and 
[here](https://github.com/apache/arrow/blob/master/format/Schema.fbs#L516)).
   * C data interface: list of key pairs 
[here](https://arrow.apache.org/docs/format/CDataInterface.html#c.ArrowSchema.metadata).
   
   I suspect that in both cases they are meant to represent _unique_ keys 
without a specific order, but it is not clear to me atm.
   
   imo it is very much worth investigating if we can get away with a common 
representation under the integration tests - as you wrote, much easier to 
reason about.


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