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

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

shangxinli commented on PR #1014:
URL: https://github.com/apache/parquet-mr/pull/1014#issuecomment-1407713001

   Thanks @wgtmac for working on this and Thanks @gszadovszky and @ggershinsky 
for reviewing it! I am a little late for the comments discussion but I see we 
are in the right direction. Let's address it in a separate discussion. If it 
turns out that changing the parquet-format is the right way to solve it, we can 
make the proposal and I can help for the approval. 
   
   My comments are all addressed. I don't have further comments. 




> Refactor different file rewriters to use single implementation
> --------------------------------------------------------------
>
>                 Key: PARQUET-2227
>                 URL: https://issues.apache.org/jira/browse/PARQUET-2227
>             Project: Parquet
>          Issue Type: Sub-task
>          Components: parquet-mr
>            Reporter: Gang Wu
>            Assignee: Gang Wu
>            Priority: Major
>
> A new ParquetRewriter is implemented to support all logics in the 
> ColumnPruner, CompressionConverter, ColumnMasker, and ColumnEncrypter. And 
> refactor all the old rewriters to use ParquetRewriter under the hood.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to