adamreeve opened a new issue, #16777: URL: https://github.com/apache/datafusion/issues/16777
Follow up task to #16649 Working with encrypted Parquet files is a fairly niche feature in my opinion, so it seems like this feature should not be enabled by default. That said, it looks like DataFusion already depends on the `ring` crate, which is the main extra dependency required for this feature, via `object_store` and `aws-config`. So disabling this by default might not make a huge difference to compile times and binary sizes. Notes from @alamb on what would be required to make this non-default: 1. Don't add it to the default features 2. Update the docs to mention the config setting requires the `parquet_encryption` feature -- https://github.com/apache/datafusion/blob/d66d6b9eb83a4643bc8d1ee9c2da7f0fa644c0cf/datafusion/common/src/config.rs#L1839 3. Enable it in datafusion-cli (add `parquet_encryption` in https://github.com/apache/datafusion/blob/ca16255e725bf6676a8ddd4b9948496d99b5bd88/datafusion-cli/Cargo.toml#L45-L44) -- 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.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
