rdblue commented on code in PR #2639: URL: https://github.com/apache/iceberg/pull/2639#discussion_r1155355157
########## parquet/src/main/java/org/apache/iceberg/parquet/ParquetWriter.java: ########## @@ -76,12 +58,14 @@ private final OutputFile output; private final Configuration conf; - private DynMethods.BoundMethod flushPageStoreToWriter; + private ColumnChunkPageWriteStore pageStore = null; private ColumnWriteStore writeStore; private long recordCount = 0; private long nextCheckRecordCount = 10; private boolean closed; private ParquetFileWriter writer; + private final InternalFileEncryptor fileEncryptor; Review Comment: This should probably be relocated to the block of final instance fields. -- 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
