Jefffrey commented on code in PR #11209:
URL: https://github.com/apache/arrow-rs/pull/11209#discussion_r4167697653
##########
arrow-data/src/transform/mod.rs:
##########
@@ -843,6 +843,16 @@ impl<'a> MutableArrayData<'a> {
));
}
+ if len == 0 {
+ return Ok(());
+ }
+
+ if matches!(&self.data.data_type, DataType::Union(fields, _) if
fields.is_empty()) {
Review Comment:
im sorry but im just having a hard time believing this isnt purely LLM
output. how could that original line of `cargo validation could not run locally
because Cargo is not installed.` pass a human review and not at least be a
warning flag to them that something is awry?
not to mention this just reads as LLM to me:
> I will keep future replies focused on the concrete code change and
reproducible command output.
--
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]
