friendlymatthew commented on PR #7914: URL: https://github.com/apache/arrow-rs/pull/7914#issuecomment-3078243701
Hi, round tripping variant objects would fail because we were writing out object field names by the sort order. This is fine for variants with a sorted dictionary, but everything else will produce inconsistent results What we want instead is to write out the object fields by insertion order, we can do this by passing in the `VariantMetadata` ahead of time. This way we know the insertion order of field names I'm sure we can improve the implementation, but it is correct cc @alamb -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
