ggershinsky commented on pull request #808: URL: https://github.com/apache/parquet-mr/pull/808#issuecomment-672750035
@shangxinli Indeed, my understanding of this pull request is accurately described by "'how to transport the settings from the extended ParquetWriteSupport to CryptoPropertiesFactory implementation". Since both objects live in the same process, and the PR transport is designed for pass pointers from one to the other, RPC or file loading cannot serve as additional transport channels for this. As for other points in your comment, I also agree we are starting to circle back in this discussion; I think I've provided all input I had for now, both technical and the concerns. Having a call would not be optimal, as we need to ensure transparency of PR discussions. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
