adamreeve commented on code in PR #17342:
URL: https://github.com/apache/datafusion/pull/17342#discussion_r2308836875
##########
datafusion/datasource-parquet/src/opener.rs:
##########
@@ -152,16 +152,18 @@ impl FileOpener for ParquetOpener {
let mut predicate_file_schema = Arc::clone(&self.logical_file_schema);
let mut enable_page_index = self.enable_page_index;
- let file_decryption_properties =
- self.get_file_decryption_properties(file_location)?;
-
- // For now, page index does not work with encrypted files. See:
- // https://github.com/apache/arrow-rs/issues/7629
- if file_decryption_properties.is_some() {
- enable_page_index = false;
- }
+ let encryption_context = self.get_encryption_context();
Ok(Box::pin(async move {
+ let file_decryption_properties = encryption_context
+ .get_file_decryption_properties(&file_location)
+ .await?;
+ // For now, page index does not work with encrypted files. See:
+ // https://github.com/apache/arrow-rs/issues/7629
Review Comment:
I noticed it should be possible to remove this workaround now. I've made
#17352 to follow up on this as it should be a separate change
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
