abacef commented on code in PR #7945:
URL: https://github.com/apache/arrow-rs/pull/7945#discussion_r2212324433
##########
parquet-variant-compute/src/variant_array.rs:
##########
@@ -88,7 +90,10 @@ impl VariantArray {
));
};
// Ensure the StructArray has a metadata field of BinaryView
- let Some(metadata_field) = inner.fields().iter().find(|f| f.name() ==
"metadata") else {
+
+ let cloned = inner.clone();
Review Comment:
When doing a clone, is it just incrementing the reference count to the
function parameter? If so, maybe finding the metadata reference and the value
reference on either the passed in StructArray or the clone will come up with
the same value?
--
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]
