felixYyu edited a comment on pull request #4439: URL: https://github.com/apache/iceberg/pull/4439#issuecomment-1085297114
> description with the motivation and what you're actually changing? - DynClasses DynFields SnapshotSummary : A Builder with a static factory method on the encapsulating class must have a private constructor - BaseFileScanTask : This method returns both mutable and immutable collections or maps from different paths,These grouping parentheses are unnecessary; it is unlikely the code will be misinterpreted without them. - FlinkCatalog : This method returns both mutable and immutable collections or maps from different paths - ParquetMetricsRowGroupFilter : Integer id = ref.fieldId(); id type maybe privimitive -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
