You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@parquet.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2023/01/14 16:21:00 UTC

[jira] [Commented] (PARQUET-2227) Refactor different file rewriters to use single implementation

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

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

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

   > I am afraid some implementations may drop characters after `'\n'` when displaying the string content. Let me do some investigation.
   
   I do not have a strong opinion for `'\n'` only that we need a character that probably won't be used by any systems writing parquet files.




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