nk1506 commented on issue #10742:
URL: https://github.com/apache/iceberg/issues/10742#issuecomment-2247718623

   IMO, we should add the validation for null check at least. Else it may 
violate the the table definition constraints. Having extra columns to parquet 
is being ignored from column metrics. Also after rewrite_data_file those extra 
columns will be removed. Adding parquet files with null values are always being 
considered. It can give wrong results for ZERO count. 
   WDYT? 


-- 
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

Reply via email to