rust/greptimedb
1
0
Fork 0
mirror of https://github.com/GreptimeTeam/greptimedb.git synced 2026-05-14 03:50:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: incorrect prefilter check (#7886)

Browse Source 
Signed-off-by: evenyag <realevenyag@gmail.com>
This commit is contained in:
Yingwen
2026-03-31 10:45:07 +08:00
committed by GitHub
parent fb2dd862d5
commit dde1edcdb4
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/mito2/src/sst/parquet/flat_format.rs
View File

@@ -282,10 +282,10 @@ impl FlatReadFormat {
}
}
/// Returns `true` if raw batches from parquet use the flat layout with a
/// dictionary-encoded `__primary_key` column (i.e., [`ParquetAdapter::Flat`]).
/// Returns `true` if raw batches from parquet use the flat layout and
/// stores primary key columns as raw columns.
/// Returns `false` for the legacy primary-key-to-flat conversion path.
pub(crate) fn raw_batch_has_primary_key_dictionary(&self) -> bool {
pub(crate) fn batch_has_raw_pk_columns(&self) -> bool {
matches!(&self.parquet_adapter, ParquetAdapter::Flat(_))
}

4
src/mito2/src/sst/parquet/prefilter.rs
View File

@@ -217,9 +217,9 @@ impl PrefilterContextBuilder {
return None;
}
// Only flat format with dictionary-encoded PKs supports PK prefiltering.
// Only perform PK prefiltering for primary-key-to-flat conversion path.
let flat_format = read_format.as_flat()?;
if !flat_format.raw_batch_has_primary_key_dictionary() {
if flat_format.batch_has_raw_pk_columns() {
return None;
}