tolleybot commented on code in PR #34616:
URL: https://github.com/apache/arrow/pull/34616#discussion_r1349241809
##########
cpp/src/arrow/dataset/parquet_encryption_config.h:
##########
@@ -29,44 +29,50 @@ struct DecryptionConfiguration;
namespace arrow {
namespace dataset {
-/// \brief Core configuration class encapsulating parameters for high-level
encryption
-/// within Parquet framework.
-///
-/// ParquetEncryptionConfig serves as a bridge, passing encryption-related
-/// parameters to appropriate components within the Parquet library. It holds
references
-/// to objects defining encryption strategy, Key Management Service (KMS)
configuration,
-/// and specific encryption configurations for Parquet data.
-///
-/// \member crypto_factory Shared pointer to CryptoFactory object, responsible
for
-/// creating cryptographic components like encryptors and decryptors. \member
-/// kms_connection_config Shared pointer to KmsConnectionConfig object, holding
-/// configuration parameters for connecting to a Key Management Service (KMS).
-/// \member encryption_config Shared pointer to EncryptionConfiguration
object, defining
-/// specific encryption settings for Parquet data, like keys for different
columns.
struct ARROW_DS_EXPORT ParquetEncryptionConfig {
std::shared_ptr<parquet::encryption::CryptoFactory> crypto_factory;
std::shared_ptr<parquet::encryption::KmsConnectionConfig>
kms_connection_config;
std::shared_ptr<parquet::encryption::EncryptionConfiguration>
encryption_config;
+ /// \brief Core configuration class encapsulating parameters for high-level
encryption
Review Comment:
@anjakefala do I need to do anything with this or are you handling it?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
