rok commented on code in PR #7111:
URL: https://github.com/apache/arrow-rs/pull/7111#discussion_r1999681758


##########
parquet/src/encryption/encrypt.rs:
##########
@@ -152,8 +152,20 @@ impl EncryptionPropertiesBuilder {
     /// Set the key used for encryption of a column. Note that if no column 
keys are provided but
     /// footer key is all columns will be encrypted with the footer key. If 
column keys are provided
     /// only the columns with a key will be encrypted even if footer key is 
provided.
-    pub fn with_column_key(mut self, column_path: String, encryption_key: 
EncryptionKey) -> Self {
-        self.column_keys.insert(column_path, encryption_key);
+    pub fn with_column_key(mut self, column_name: &str, encryption_key: 
EncryptionKey) -> Self {
+        self.column_keys.insert(column_name.to_string(), encryption_key);
+        self
+    }

Review Comment:
   Done.



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to