phillipleblanc opened a new pull request, #10059:
URL: https://github.com/apache/arrow-datafusion/pull/10059

   ## Which issue does this PR close?
   
   Closes #10058
   
   ## Rationale for this change
   
   Makes it possible to actually disable the crypto features if they aren't 
required for a user of DataFusion.
   
   ## What changes are included in this PR?
   
   In addition to the main change that fixes the issue above, I realized that 
the `crypto_expressions` feature was added in two other crates that aren't 
using it anymore - so I removed them.
   
   ## Are these changes tested?
   
   I tested that building with no default features does actually disable the 
crypto dependencies.
   
   ## Are there any user-facing changes?
   
   This could be a breaking change if people disabled the default features and 
were still using a crypto function. I will lean on the more experience 
contributors to weigh in here.


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

Reply via email to