Vyquos opened a new pull request, #17700:
URL: https://github.com/apache/datafusion/pull/17700

   ## Which issue does this PR close?
   
   This PR adds to #16649 . That PR separates the `parquet` feature into 
`parquet` and `parquet_encryption`, making encryption optional. As with that 
PR, this one also relates to issue #16650 .
   
   ## Rationale for this change
   
   Even with the previous PR, `parquet/encryption` remains enabled because it 
is set in the top-level `Cargo.toml`. The PR fixes this.
   This PR ensures that the `parquet/encryption` feature isn't enabled by 
default.
   
   ## Are these changes tested?
   
   I'm honestly not fully familiar with datafusion tests relating to the 
parquet encryption feature. I believe that no further adjustment to existing 
tests or creation of new tests is required since this is a very minor change.
   
   ## Are there any user-facing changes?
   
   Same as for #16649 , no user-facing changes.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to