rdblue commented on code in PR #2639:
URL: https://github.com/apache/iceberg/pull/2639#discussion_r1133145246
##########
parquet/src/main/java/org/apache/iceberg/parquet/ParquetWriter.java:
##########
@@ -55,6 +57,9 @@
CodecFactory.BytesCompressor.class,
MessageType.class,
ByteBufferAllocator.class,
+ int.class,
+ boolean.class,
+ InternalFileEncryptor.class,
Review Comment:
Looks like `ColumnChunkPageWriteStore` has been public since this commit:
https://github.com/apache/parquet-mr/commit/89c5c25d44f8959d1a66493c06630ab0060cf346
That's in all 1.12.x releases, so I think we can remove the use of
reflection.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
