fix: check and clippy

Signed-off-by: shuiyisong <xixing.sys@gmail.com>
This commit is contained in:
shuiyisong
2025-11-26 18:35:21 +08:00
parent dd043eadc4
commit 59ddfa84ec
5 changed files with 12 additions and 24 deletions

View File

@@ -112,8 +112,7 @@ pub trait InvertedIndexReader: Send + Sync {
ranges: &[Range<u64>],
metrics: Option<&'a mut InvertedIndexReadMetrics>,
) -> Result<Vec<FstMap>> {
let mut metrics = metrics;
self.read_vec(ranges, metrics.as_deref_mut())
self.read_vec(ranges, metrics)
.await?
.into_iter()
.map(|bytes| FstMap::new(bytes.to_vec()).context(DecodeFstSnafu))
@@ -141,9 +140,8 @@ pub trait InvertedIndexReader: Send + Sync {
ranges: &[(Range<u64>, BitmapType)],
metrics: Option<&'a mut InvertedIndexReadMetrics>,
) -> Result<VecDeque<Bitmap>> {
let mut metrics = metrics;
let (ranges, types): (Vec<_>, Vec<_>) = ranges.iter().cloned().unzip();
let bytes = self.read_vec(&ranges, metrics.as_deref_mut()).await?;
let bytes = self.read_vec(&ranges, metrics).await?;
bytes
.into_iter()
.zip(types)

View File

@@ -57,9 +57,9 @@ impl<R> InvertedIndexFooterReader<R> {
}
impl<R: RangeReader> InvertedIndexFooterReader<R> {
pub async fn metadata<'a>(
pub async fn metadata(
&mut self,
mut metrics: Option<&'a mut InvertedIndexReadMetrics>,
mut metrics: Option<&mut InvertedIndexReadMetrics>,
) -> Result<InvertedIndexMetas> {
ensure!(
self.blob_size >= FOOTER_PAYLOAD_SIZE_SIZE,

View File

@@ -33,12 +33,11 @@ impl<'a> ParallelFstValuesMapper<'a> {
Self { reader }
}
pub async fn map_values_vec<'b>(
pub async fn map_values_vec(
&mut self,
value_and_meta_vec: &[(Vec<u64>, &'b InvertedIndexMeta)],
value_and_meta_vec: &[(Vec<u64>, &InvertedIndexMeta)],
metrics: Option<&mut InvertedIndexReadMetrics>,
) -> Result<Vec<Bitmap>> {
let mut metrics = metrics;
let groups = value_and_meta_vec
.iter()
.map(|(values, _)| values.len())
@@ -66,10 +65,7 @@ impl<'a> ParallelFstValuesMapper<'a> {
}
common_telemetry::debug!("fetch ranges: {:?}", fetch_ranges);
let mut bitmaps = self
.reader
.bitmap_deque(&fetch_ranges, metrics.as_deref_mut())
.await?;
let mut bitmaps = self.reader.bitmap_deque(&fetch_ranges, metrics).await?;
let mut output = Vec::with_capacity(groups.len());
for counter in groups {

View File

@@ -94,9 +94,7 @@ impl IndexApplier for PredicatesIndexApplier {
.collect::<Vec<_>>();
let mut mapper = ParallelFstValuesMapper::new(reader);
let mut bm_vec = mapper
.map_values_vec(&value_and_meta_vec, metrics.as_deref_mut())
.await?;
let mut bm_vec = mapper.map_values_vec(&value_and_meta_vec, metrics).await?;
let mut bitmap = bm_vec.pop().unwrap(); // SAFETY: `fst_ranges` is not empty
for bm in bm_vec {

View File

@@ -615,11 +615,9 @@ impl IndexSource {
let (reader, fallbacked) = self.ensure_reader(file_id, file_size_hint).await?;
// Track cache miss if fallbacked to remote
if fallbacked {
if let Some(m) = metrics {
if fallbacked && let Some(m) = metrics {
m.blob_cache_miss += 1;
}
}
let res = reader.blob(key).await;
match res {
@@ -655,11 +653,9 @@ impl IndexSource {
let (reader, fallbacked) = self.ensure_reader(file_id, file_size_hint).await?;
// Track cache miss if fallbacked to remote
if fallbacked {
if let Some(m) = &mut metrics {
if fallbacked && let Some(m) = &mut metrics {
m.blob_cache_miss += 1;
}
}
let start = metrics.as_ref().map(|_| Instant::now());
let res = reader.dir(key).await;