rok commented on PR #9532:
URL: https://github.com/apache/arrow-rs/pull/9532#issuecomment-4034665999
Hey :wave:. Primary motivation for making `with_file_decryption_properties`
`pub(crate)` was to make API easier to refactor (there was also an upcoming
refactor of parquet that I think had already happened). So I don't see an issue
with `pub` here. Also `with_file_{en,de}cryption_properties` API is pretty
locked in at this point.
One general thought is that we'd want to limit footguns by steering people
towards KMSes and away from directly handling keys. We did that in PyArrow and
it turned out not very practical. So consider that, but feel free to ignore it.
--
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]
