itamarst commented on a change in pull request #9631:
URL: https://github.com/apache/arrow/pull/9631#discussion_r610621630



##########
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:
       Are you working on Arrow in your spare time? Or the Ursa Labs people? 
The main driver for all of this is companies that want to use these tools, and 
funding it one way or another. That being said, I mention they're client of 
mine as, it's true, a caveat: I can't speak to their particular technical 
assessment of the high-level API, but they did make one.
   
   If you're willing I can try to introduce you to the people who decided the 
high-level API wasn't worth using from their perspective, and maybe you can 
convince them they're wrong.




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


Reply via email to