Skip to content

Commit 100bf47

Browse files
authored
chore(query): remove SliceExt (#16853)
* chore(query): remove SliceExt * chore(query): remove SliceExt
1 parent ceb2d03 commit 100bf47

File tree

4 files changed

+4
-69
lines changed

4 files changed

+4
-69
lines changed

src/common/base/src/lib.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ pub mod http_client;
3636
pub mod mem_allocator;
3737
pub mod rangemap;
3838
pub mod runtime;
39-
pub mod slice_ext;
4039
pub mod vec_ext;
4140
pub mod version;
4241

src/common/base/src/slice_ext.rs

Lines changed: 0 additions & 62 deletions
This file was deleted.

src/query/expression/src/kernels/take.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ use databend_common_arrow::arrow::array::Array;
1919
use databend_common_arrow::arrow::array::Utf8ViewArray;
2020
use databend_common_arrow::arrow::bitmap::Bitmap;
2121
use databend_common_arrow::arrow::buffer::Buffer;
22-
use databend_common_base::slice_ext::GetSaferUnchecked;
2322
use databend_common_exception::Result;
2423
use string::StringColumnBuilder;
2524

@@ -258,7 +257,7 @@ where I: databend_common_arrow::arrow::types::Index
258257
let result: Vec<T> = self
259258
.indices
260259
.iter()
261-
.map(|index| unsafe { *col.get_unchecked_release(index.to_usize()) })
260+
.map(|index| unsafe { *col.get_unchecked(index.to_usize()) })
262261
.collect();
263262
result.into()
264263
}

src/query/expression/src/types/string.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ use std::ops::Range;
1818
use databend_common_arrow::arrow::array::MutableBinaryViewArray;
1919
use databend_common_arrow::arrow::array::Utf8ViewArray;
2020
use databend_common_arrow::arrow::trusted_len::TrustedLen;
21-
use databend_common_base::slice_ext::GetSaferUnchecked;
2221
use databend_common_exception::ErrorCode;
2322
use databend_common_exception::Result;
2423

@@ -293,8 +292,8 @@ impl StringColumn {
293292
}
294293

295294
pub fn compare(col_i: &Self, i: usize, col_j: &Self, j: usize) -> Ordering {
296-
let view_i = unsafe { col_i.data.views().as_slice().get_unchecked_release(i) };
297-
let view_j = unsafe { col_j.data.views().as_slice().get_unchecked_release(j) };
295+
let view_i = unsafe { col_i.data.views().as_slice().get_unchecked(i) };
296+
let view_j = unsafe { col_j.data.views().as_slice().get_unchecked(j) };
298297

299298
if view_i.prefix == view_j.prefix {
300299
unsafe {
@@ -311,7 +310,7 @@ impl StringColumn {
311310
}
312311

313312
pub fn compare_str(col: &Self, i: usize, value: &str) -> Ordering {
314-
let view = unsafe { col.data.views().as_slice().get_unchecked_release(i) };
313+
let view = unsafe { col.data.views().as_slice().get_unchecked(i) };
315314
let prefix = load_prefix(value.as_bytes());
316315

317316
if view.prefix == prefix {

0 commit comments

Comments
 (0)