adamreeve commented on PR #7387: URL: https://github.com/apache/arrow-rs/pull/7387#issuecomment-2819908130
I raised this issue at the recent Parquet sync (see notes at https://lists.apache.org/thread/qd6kd13mv19xcfkwn19vzlnd4zgxmyf3) and agreed with @alamb that this doesn't need to be part of arrow-rs and should instead be a standalone third-party crate, particularly as this isn't part of the official Parquet specification. I'll most-likely set up a repository for this in the @G-Research org. That doesn't rule out the possibility of integrating this into arrow-rs later if there's enough community demand for more official support for this feature, and available maintenance capacity. One loose end from this PR is the question of async KMS clients. That would require changes to arrow-rs, in particular adding an async version of the [`KeyRetriever`](https://github.com/apache/arrow-rs/blob/6a6c63123130f28c6f42544849bdebeb3e36531f/parquet/src/encryption/decrypt.rs#L102) trait. That's not a high priority for me right now but is something I'll try to come back to and address. -- 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]
