[ 
https://issues.apache.org/jira/browse/PARQUET-2081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17423952#comment-17423952
 ] 

ASF GitHub Bot commented on PARQUET-2081:
-----------------------------------------

shangxinli edited a comment on pull request #928:
URL: https://github.com/apache/parquet-mr/pull/928#issuecomment-933499949


   @gszadovszky Just addressed your feedback. For the conflict, I tried to 
merge from upstream but it always resulted in a lot more conflict. So I just 
simply revert my change that partially renamed AAD to pageHeaderAAD. I looked 
at the util.java file and I see there are a lot of other places all using AAD 
instead of a more specific AAD. We can have another PR  to fix them all. 


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


> Encryption translation tool - Parquet-hadoop
> --------------------------------------------
>
>                 Key: PARQUET-2081
>                 URL: https://issues.apache.org/jira/browse/PARQUET-2081
>             Project: Parquet
>          Issue Type: Task
>          Components: parquet-mr
>            Reporter: Xinli Shang
>            Priority: Major
>             Fix For: 1.13.0
>
>
> This is the implement the core part of the Encryption translation tool in 
> parquet-hadoop. After this, we will have another Jira/PR for parquet-cli to 
> integrate with key tools for encryption properties.. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to