ggershinsky commented on a change in pull request #9631:
URL: https://github.com/apache/arrow/pull/9631#discussion_r610661538
##########
File path: cpp/src/parquet/encryption/encryption.h
##########
@@ -70,6 +70,26 @@ class PARQUET_EXPORT StringKeyIdRetriever : public
DecryptionKeyRetriever {
std::map<std::string, std::string> key_map_;
};
+// Function variant of DecryptionKeyRetriever, taking a state object.
Review comment:
I work on parquet encryption in a number of OSS repos/frameworks, trying
to make sure these frameworks are interoperable regardless of the ecosystem
they belong to. Also, trying to make the use of this security tool as safe as
possible. The high-level layer addresses these goals; it is unfortunate that
your client has decided it is not worth using. There are quite a few companies
that made a different decision :)
--
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]
