shangxinli commented on a change in pull request #928:
URL: https://github.com/apache/parquet-mr/pull/928#discussion_r716877124
##########
File path: pom.xml
##########
@@ -522,6 +522,9 @@
change to fix a integer overflow issue.
TODO: remove this after Parquet 1.13 release -->
<exclude>org.apache.parquet.column.values.dictionary.DictionaryValuesWriter#dictionaryByteSize</exclude>
+
<exclude>org.apache.parquet.hadoop.util.ColumnMasker.MaskMode</exclude>
+
<exclude>org.apache.parquet.hadoop.util.CompressionConverter</exclude>
+ <exclude>org.apache.parquet.hadoop.util.ColumnMasker</exclude>
Review comment:
@gszadovszky, Sorry my bad. Making it private won't work because it is
still used outside util package. So I just revert the refactoring code to avoid
the exclusion here. We can do it later.
--
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]
