Skip to content

Commit

Permalink
feat:Add function for row alignment with page mask (#1791)
Browse files Browse the repository at this point in the history
* add range and rowRanges

* add some tests for range

* add filter logic

* fix fmt

* fix todo

* fix test

* Apply suggestions from code review

Co-authored-by: Liang-Chi Hsieh <viirya@gmail.com>

* fix compute_row_ranges

* fix annotation

* change to use std:ops:RangeInclusive

* Apply suggestions from code review

Co-authored-by: Raphael Taylor-Davies <1781103+tustvold@users.noreply.github.com>

* fix

* Update parquet/src/file/page_index/range.rs

Co-authored-by: Raphael Taylor-Davies <1781103+tustvold@users.noreply.github.com>

Co-authored-by: Liang-Chi Hsieh <viirya@gmail.com>
Co-authored-by: Raphael Taylor-Davies <1781103+tustvold@users.noreply.github.com>
  • Loading branch information
3 people committed Jun 6, 2022
1 parent c8d4323 commit 9b220a5
Show file tree
Hide file tree
Showing 4 changed files with 477 additions and 0 deletions.
1 change: 1 addition & 0 deletions parquet/src/file/metadata.rs
Original file line number Diff line number Diff line change
Expand Up @@ -223,6 +223,7 @@ pub struct RowGroupMetaData {
num_rows: i64,
total_byte_size: i64,
schema_descr: SchemaDescPtr,
// Todo add filter result -> row range
}

impl RowGroupMetaData {
Expand Down
1 change: 1 addition & 0 deletions parquet/src/file/page_index/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,4 @@

pub mod index;
pub mod index_reader;
pub(crate) mod range;

0 comments on commit 9b220a5

Please sign in to comment.