ggershinsky commented on a change in pull request #9631:
URL: https://github.com/apache/arrow/pull/9631#discussion_r609399218
##########
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:
is a change in the low level C++ layer needed to build a wrapper layer
on top of it? for example, we have the #8023 that also builds a layer on top,
but without additions to low level layer (besides some light fixes), working
with the DecryptionKeyRetriever as it is today.
Since this pull request is a temporary python wrapping for decryption,
helpful until the high-level python wrapper is ready, not sure it makes sense
to introduce changes in the low level layer in order to accommodate this pr.
--
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]
