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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
