[GitHub] [incubator-iceberg] rdblue commented on issue #173: Make collection of lower/upper bounds configurable

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

rdblue commented on issue #173: Make collection of lower/upper bounds configurable
URL: https://github.com/apache/incubator-iceberg/issues/173#issuecomment-498070912
 
 
   I'd prefer the simpler table property approach because it can be configured using existing SQL DDL (`SET TBLPROPERTIES`). Here's what I would do to keep the configuration small:
   
   1. Define modes: `full`, `counts`, and `none` to express all metrics, just null/value counts, and no metrics
   2. Allow setting a column default: `metadata.metrics.default=full`
   3. Add column-level config: `metadata.metrics.column.(column-name)=counts` to allow per-column tuning
   
   That way, the configuration stays simple and users can choose how to minimize the amount of configuration properties they need to set by controlling the default and then updating individual columns from there.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

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