This is an automated email from the ASF dual-hosted git repository.
agrove pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git
The following commit(s) were added to refs/heads/master by this push:
new 1d4a928eb Fix the test failure of `size_of_scalar` failure on M1/M2
Mac (#3398)
1d4a928eb is described below
commit 1d4a928eb76627ed4f0923f1cae89cda6a9c035a
Author: Remzi Yang <[email protected]>
AuthorDate: Tue Sep 13 02:19:05 2022 +0800
Fix the test failure of `size_of_scalar` failure on M1/M2 Mac (#3398)
* fix
Signed-off-by: remzi <[email protected]>
* always 48
Signed-off-by: remzi <[email protected]>
Signed-off-by: remzi <[email protected]>
---
datafusion/common/src/scalar.rs | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/datafusion/common/src/scalar.rs b/datafusion/common/src/scalar.rs
index be06f7b3a..6c60bc373 100644
--- a/datafusion/common/src/scalar.rs
+++ b/datafusion/common/src/scalar.rs
@@ -2575,12 +2575,7 @@ mod tests {
// The alignment requirements differ across architectures and
// thus the size of the enum appears to as as well
- let expected = match cfg!(target_arch = "aarch64") {
- true => 64,
- false => 48,
- };
-
- assert_eq!(std::mem::size_of::<ScalarValue>(), expected);
+ assert_eq!(std::mem::size_of::<ScalarValue>(), 48);
}
#[test]
