[GitHub] [iceberg] szehon-ho commented on pull request #5215: Core: Update MetricsConfig to use a default for first 32 columns

[GitBox <gi...@apache.org>]

szehon-ho commented on PR #5215:
URL: https://github.com/apache/iceberg/pull/5215#issuecomment-1180914739

   I think that's an interesting enhancement idea, if I understand, to make write.metadata.metrics.max-inferred-column-defaults to also accept first or last.  An organization might be able to set last(32) as default on the catalog level (ie, https://github.com/apache/iceberg/pull/4011).  On the other hand, sort and partition columns are already auto-promoted to have stats today, curious do you see a good impact of having stats on other columns?


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org