Skip to content

Commit

Permalink
refactor: Remove unused Parquet indexes (#18329)
Browse files Browse the repository at this point in the history
  • Loading branch information
coastalwhite authored Aug 23, 2024
1 parent fea6b02 commit c37bad2
Show file tree
Hide file tree
Showing 32 changed files with 46 additions and 1,944 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -183,11 +183,9 @@ impl<'a, D: Decoder> State<'a, D> {

pub fn not_implemented(page: &DataPage) -> ParquetError {
let is_optional = page.descriptor.primitive_type.field_info.repetition == Repetition::Optional;
let is_filtered = page.selected_rows().is_some();
let required = if is_optional { "optional" } else { "required" };
let is_filtered = if is_filtered { ", index-filtered" } else { "" };
ParquetError::not_supported(format!(
"Decoding {:?} \"{:?}\"-encoded {required}{is_filtered} parquet pages not yet supported",
"Decoding {:?} \"{:?}\"-encoded {required} parquet pages not yet supported",
page.descriptor.primitive_type.physical_type,
page.encoding(),
))
Expand Down
44 changes: 0 additions & 44 deletions crates/polars-parquet/src/arrow/read/indexes/binary.rs

This file was deleted.

20 changes: 0 additions & 20 deletions crates/polars-parquet/src/arrow/read/indexes/boolean.rs

This file was deleted.

70 changes: 0 additions & 70 deletions crates/polars-parquet/src/arrow/read/indexes/fixed_len_binary.rs

This file was deleted.

Loading

0 comments on commit c37bad2

Please sign in to comment.