friendlymatthew commented on code in PR #7943: URL: https://github.com/apache/arrow-rs/pull/7943#discussion_r2215455919
########## parquet-variant/src/variant/metadata.rs: ########## @@ -332,6 +334,30 @@ impl<'m> VariantMetadata<'m> { } } +// According to the spec, metadata dictionaries are not required to be in a specific order, +// to enable flexibility when constructing Variant values +// +// Instead of comparing the raw bytes of 2 variant metadata instances, this implementation +// checks whether the dictionary entries are equal -- regardless of their sorting order +impl<'m> PartialEq for VariantMetadata<'m> { + fn eq(&self, other: &Self) -> bool { Review Comment: Very nice. I will think about this and push up a PR. -- 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