jorgecarleitao commented on a change in pull request #9339:
URL: https://github.com/apache/arrow/pull/9339#discussion_r565522791
##########
File path: rust/arrow/src/array/transform/mod.rs
##########
@@ -714,6 +714,48 @@ mod tests {
assert_eq!(array, expected)
}
+ #[test]
+ fn test_struct_offset() {
+ let strings: ArrayRef = Arc::new(StringArray::from(vec![
+ Some("joe"),
+ None,
+ None,
+ Some("mark"),
+ Some("doe"),
+ ]));
+ let ints: ArrayRef = Arc::new(Int32Array::from(vec![
+ Some(1),
+ Some(2),
+ Some(3),
+ Some(4),
+ Some(5),
+ ]));
+
+ let array =
+ StructArray::try_from(vec![("f1", strings.clone()), ("f2",
ints.clone())])
+ .unwrap()
+ .slice(1, 3)
+ .data();
+ let arrays = vec![array.as_ref()];
+ let mut mutable = MutableArrayData::new(arrays, false, 0);
+
+ mutable.extend(0, 1, 3);
+ let data = mutable.freeze();
+ let array = StructArray::from(Arc::new(data));
+
+ // Struct equality doesn't seem to work when using slices?
Review comment:
I am sorry, I was not very specific. There are two different issues: for
string arrays, binary, etc., I think that the issue is solved with #9211, via
https://github.com/apache/arrow/pull/9211/files#diff-74d8df3798e724950c2eb5aae1a838fc2f2b9e35d89ace2627e8e7a64584c71fR91
For the Struct, we need another patch.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]