corwinjoy commented on PR #16649:
URL: https://github.com/apache/datafusion/pull/16649#issuecomment-3026163921

   ## Pull Request Overview from Copilot
   
   This PR introduces an optional `parquet_encryption` feature that wires 
encryption/decryption properties through the Parquet reader and writer layers 
and updates workspace dependencies.
   
   - Gate encryption/decryption logic behind `parquet_encryption` in 
`source.rs`, `opener.rs`, and `file_format.rs`
   - Expose and stub `FileEncryptionProperties`/`FileDecryptionProperties` 
types in `common/src/config.rs`
   - Add optional `hex` crate and define the `parquet_encryption` feature in 
multiple `Cargo.toml` files
   
   
   <details>
   <summary>Show a summary per file</summary>
   
   | File                                                         | Description 
                                                     |
   | ------------------------------------------------------------ | 
---------------------------------------------------------------- |
   | datafusion/datasource-parquet/src/source.rs                  | Fallback 
`file_decryption_properties` when encryption is off     |
   | datafusion/datasource-parquet/src/opener.rs                  | 
Conditionally pass decryption props to `ArrowReaderOptions`      |
   | datafusion/datasource-parquet/src/file_format.rs             | Apply cfg 
guards around decryption props for schema/statistics   |
   | datafusion/datasource-parquet/Cargo.toml                     | Add 
optional `hex` and define `parquet_encryption` feature       |
   | datafusion/core/tests/parquet/encryption.rs                  | Gate 
round-trip encryption test behind feature flag              |
   | datafusion/core/src/dataframe/parquet.rs                     | Gate 
dataframe encryption test behind feature flag               |
   | datafusion/core/Cargo.toml                                   | Enable 
`parquet_encryption` in workspace default features        |
   | datafusion/common/src/file_options/parquet_writer.rs         | 
Conditionally add encryption props in writer builder             |
   | datafusion/common/src/config.rs                              | Publicly 
expose and stub encryption types based on cfg           |
   | datafusion/common/Cargo.toml                                 | Add 
optional `hex` under `parquet_encryption`                    |
   | Cargo.toml                                                   | Introduce 
`hex` dependency (always enabled)                      |
   </details>
   
   
   


-- 
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

Reply via email to