You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ne...@apache.org on 2021/06/19 16:40:21 UTC
[arrow-rs] branch master updated: remove stale comment and update
unit tests (#472)
This is an automated email from the ASF dual-hosted git repository.
nevime pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/master by this push:
new 6e2f684 remove stale comment and update unit tests (#472)
6e2f684 is described below
commit 6e2f68420e03fe6926e8c2ffbd4441fc8cc1aeab
Author: Jiayu Liu <Ji...@users.noreply.github.com>
AuthorDate: Sun Jun 20 00:40:15 2021 +0800
remove stale comment and update unit tests (#472)
---
arrow/src/array/array_struct.rs | 24 ++----------------------
arrow/src/array/builder.rs | 24 ++----------------------
2 files changed, 4 insertions(+), 44 deletions(-)
diff --git a/arrow/src/array/array_struct.rs b/arrow/src/array/array_struct.rs
index 9c11b83..f721d35 100644
--- a/arrow/src/array/array_struct.rs
+++ b/arrow/src/array/array_struct.rs
@@ -362,28 +362,8 @@ mod tests {
.add_buffer(Buffer::from(&[1, 2, 0, 4].to_byte_slice()))
.build();
- assert_eq!(&expected_string_data, arr.column(0).data());
-
- // TODO: implement equality for ArrayData
- assert_eq!(expected_int_data.len(), arr.column(1).data().len());
- assert_eq!(
- expected_int_data.null_count(),
- arr.column(1).data().null_count()
- );
- assert_eq!(
- expected_int_data.null_bitmap(),
- arr.column(1).data().null_bitmap()
- );
- let expected_value_buf = expected_int_data.buffers()[0].clone();
- let actual_value_buf = arr.column(1).data().buffers()[0].clone();
- for i in 0..expected_int_data.len() {
- if !expected_int_data.is_null(i) {
- assert_eq!(
- expected_value_buf.as_slice()[i * 4..(i + 1) * 4],
- actual_value_buf.as_slice()[i * 4..(i + 1) * 4]
- );
- }
- }
+ assert_eq!(expected_string_data, *arr.column(0).data());
+ assert_eq!(expected_int_data, *arr.column(1).data());
}
#[test]
diff --git a/arrow/src/array/builder.rs b/arrow/src/array/builder.rs
index eacd764..66f2d81 100644
--- a/arrow/src/array/builder.rs
+++ b/arrow/src/array/builder.rs
@@ -3050,28 +3050,8 @@ mod tests {
.add_buffer(Buffer::from_slice_ref(&[1, 2, 0, 4]))
.build();
- assert_eq!(&expected_string_data, arr.column(0).data());
-
- // TODO: implement equality for ArrayData
- assert_eq!(expected_int_data.len(), arr.column(1).data().len());
- assert_eq!(
- expected_int_data.null_count(),
- arr.column(1).data().null_count()
- );
- assert_eq!(
- expected_int_data.null_bitmap(),
- arr.column(1).data().null_bitmap()
- );
- let expected_value_buf = expected_int_data.buffers()[0].clone();
- let actual_value_buf = arr.column(1).data().buffers()[0].clone();
- for i in 0..expected_int_data.len() {
- if !expected_int_data.is_null(i) {
- assert_eq!(
- expected_value_buf.as_slice()[i * 4..(i + 1) * 4],
- actual_value_buf.as_slice()[i * 4..(i + 1) * 4]
- );
- }
- }
+ assert_eq!(expected_string_data, *arr.column(0).data());
+ assert_eq!(expected_int_data, *arr.column(1).data());
}
#[test]