stevenzwu edited a comment on issue #1643: URL: https://github.com/apache/iceberg/issues/1643#issuecomment-714888988
We actually enable `objectReuse` by default so that chained operators can avoid the serialization and deserialization cost, which is huge for embarrassingly parallel DAGs. That is mainly for operator chaining. `RecordWriter` is used for network shuffle (like keyBy). Both probably shouldn't affect or dictate the object reuse semantic in source emitted records. BTW, Iceberg's `ParquetAvroValueReaders` returns a new Avro GenericRecord object for each iterator.next() call. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
