Add comment

adriangb · adriangb · commit 820cb495b3ce · 2025-04-03T12:16:17.000-05:00
diff --git a/datafusion/datasource-parquet/src/opener.rs b/datafusion/datasource-parquet/src/opener.rs
@@ -392,7 +392,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
`@@ -392,7 +392,7 @@ async fn load_page_index<T: AsyncFileReader>(`
`392`	`392`	`ArrowReaderMetadata::try_new(Arc::new(new_parquet_metadata), options)?;`
`393`	`393`	`Ok(new_arrow_reader)`
`394`	`394`	`} else {`
`395`		`- // No page index, return the original metadata`
	`395`	`+ // No need to load the page index again, just return the existing metadata`
`396`	`396`	`Ok(arrow_reader)`
`397`	`397`	`}`
`398`	`398`	`}`