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)