corwinjoy commented on code in PR #16351:
URL: https://github.com/apache/datafusion/pull/16351#discussion_r2151077910


##########
datafusion/common/src/config.rs:
##########
@@ -591,6 +930,12 @@ config_namespace! {
         /// writing out already in-memory data, such as from a cached
         /// data frame.
         pub maximum_buffered_record_batches_per_stream: usize, default = 2
+
+        /// Optional file decryption properties
+        pub file_decryption_properties: 
Option<ConfigFileDecryptionProperties>, default = None
+
+        /// Optional file encryption properties
+        pub file_encryption_properties: 
Option<ConfigFileEncryptionProperties>, default = None

Review Comment:
   @adamreeve Has a nice PR to move this all to a crypto namespace which cleans 
this up a lot. We are still debating a bit, since we want to understand the 
impact downstream for tools like delta-rs.
   https://github.com/corwinjoy/datafusion/pull/5



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