scovich commented on code in PR #7906:
URL: https://github.com/apache/arrow-rs/pull/7906#discussion_r2201726616
##########
parquet-variant/src/variant/metadata.rs:
##########
@@ -268,6 +261,16 @@ impl<'m> VariantMetadata<'m> {
"dictionary values are not unique and
ordered".to_string(),
));
}
+ } else {
+ // Validate offsets are in-bounds and monotonically increasing.
+ // Since shallow validation ensures the first and last offsets
are in bounds, we can also verify all offsets
Review Comment:
nit: strangely long line?
--
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]
