thamht4190 commented on a change in pull request #8023: URL: https://github.com/apache/arrow/pull/8023#discussion_r477067068
########## File path: cpp/src/parquet/encryption_internal.h ########## @@ -45,6 +45,9 @@ constexpr int8_t kOffsetIndex = 7; /// Performs AES encryption operations with GCM or CTR ciphers. class AesEncryptor { public: + /// Can serve one key length only. Possible values: 16, 24, 32 bytes. + explicit AesEncryptor(ParquetCipher::type alg_id, int key_len, bool metadata); Review comment: @emkornfield I want to create a local variable of AesEncryptor, without using smart pointers, so it will store in stack rather than in heap. ---------------------------------------------------------------- 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: us...@infra.apache.org