Skip to content

Commit 0d6aa5f

Browse files
committed
Column::memory_size add arg gc.
1 parent cb0f3fe commit 0d6aa5f

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/query/expression/tests/it/block.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,16 +90,16 @@ fn test_block_entry_memory_size() {
9090
assert_eq!(3, entry.memory_size());
9191

9292
let col = StringType::from_data((0..10).map(|x| x.to_string()).collect::<Vec<_>>());
93-
assert_eq!(col.memory_size(), 10 * 16);
93+
assert_eq!(col.memory_size(false), 10 * 16);
9494

9595
let array = ArrayColumn::<Int64Type>::new(
9696
Buffer::from_iter(0..10i64),
9797
Buffer::from(vec![0u64, 1, 3, 6, 10]),
9898
);
99-
let total_memory_size = array.memory_size(); // 10 * 8 + 5 * 8 = 80 + 40 = 120
99+
let total_memory_size = array.memory_size(false); // 10 * 8 + 5 * 8 = 80 + 40 = 120
100100
let expected = array
101101
.iter()
102-
.map(|x| Int64Type::column_memory_size(&x))
102+
.map(|x| Int64Type::column_memory_size(&x, false))
103103
.sum::<usize>()
104104
+ (array.len() + 1) * 8;
105105
assert_eq!(total_memory_size, expected);
@@ -108,6 +108,6 @@ fn test_block_entry_memory_size() {
108108
let array3 = array.slice(2..4);
109109
assert_eq!(
110110
total_memory_size,
111-
array2.memory_size() + array3.memory_size() - 8
111+
array2.memory_size(false) + array3.memory_size(false) - 8
112112
);
113113
}

src/query/expression/tests/it/kernel.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -625,7 +625,7 @@ fn assert_estimated_scalar_repeat_size(scalar: ScalarRef, num_rows: usize, ty: D
625625
let col = builder.build();
626626
assert_eq!(
627627
scalar.estimated_scalar_repeat_size(num_rows, &ty),
628-
col.memory_size()
628+
col.memory_size(false)
629629
);
630630
}
631631

0 commit comments

Comments
 (0)