Add comment

adriangb · adriangb · commit 05bdcf8c9af0 · 2025-04-03T12:36:40.000-05:00
diff --git a/datafusion/datasource-parquet/src/opener.rs b/datafusion/datasource-parquet/src/opener.rs
@@ -397,7 +397,7 @@ async fn load_page_index<T: AsyncFileReader>(
             ArrowReaderMetadata::try_new(Arc::new(new_parquet_metadata), options)?;
         Ok(new_arrow_reader)
     } else {
-        // No page index, return the original metadata
+        // No need to load the page index again, just return the existing metadata
         Ok(arrow_reader)
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -397,7 +397,7 @@ async fn load_page_index<T: AsyncFileReader>(`
`397`	`397`	`ArrowReaderMetadata::try_new(Arc::new(new_parquet_metadata), options)?;`
`398`	`398`	`Ok(new_arrow_reader)`
`399`	`399`	`} else {`
`400`		`- // No page index, return the original metadata`
	`400`	`+ // No need to load the page index again, just return the existing metadata`
`401`	`401`	`Ok(arrow_reader)`
`402`	`402`	`}`
`403`	`403`	`}`