From 66a59369c5d023a9d379e63aecc9b6f75d3e782f Mon Sep 17 00:00:00 2001 From: Andrew Lamb Date: Fri, 31 May 2024 08:17:08 -0400 Subject: [PATCH] improve comments --- .../src/datasource/physical_plan/parquet/access_plan.rs | 6 +++--- .../src/datasource/physical_plan/parquet/page_filter.rs | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/datafusion/core/src/datasource/physical_plan/parquet/access_plan.rs b/datafusion/core/src/datasource/physical_plan/parquet/access_plan.rs index de0d71a766743..a826c147e4253 100644 --- a/datafusion/core/src/datasource/physical_plan/parquet/access_plan.rs +++ b/datafusion/core/src/datasource/physical_plan/parquet/access_plan.rs @@ -17,10 +17,10 @@ use parquet::arrow::arrow_reader::RowSelection; -/// Specifies a selection of row groups / rows within a ParquetFile to decode. +/// Specifies a selection of rows and row groups within a ParquetFile to decode. /// -/// This structure can limit the row groups and data pages a `ParquetExec` will -/// read and decode. +/// A `ParquetAccessPlan` is used to limits the row groups and data pages a `ParquetExec` +/// will read and decode and this improve performance. /// /// Note that page level pruning based on ArrowPredicate is applied after all of /// these selections diff --git a/datafusion/core/src/datasource/physical_plan/parquet/page_filter.rs b/datafusion/core/src/datasource/physical_plan/parquet/page_filter.rs index 4cece1981b7f1..d2ec4b0d0c698 100644 --- a/datafusion/core/src/datasource/physical_plan/parquet/page_filter.rs +++ b/datafusion/core/src/datasource/physical_plan/parquet/page_filter.rs @@ -284,7 +284,7 @@ fn find_column_index( /// Intersects the [`RowSelector`]s /// -/// For exampe, given: +/// For example, given: /// * `RowSelector1: [ Skip(0~199), Read(200~299)]` /// * `RowSelector2: [ Skip(0~99), Read(100~249), Skip(250~299)]` ///