aokolnychyi commented on issue #173: Make collection of lower/upper bounds configurable URL: https://github.com/apache/incubator-iceberg/issues/173#issuecomment-498022864 I am wondering about the best way to implement this feature. A few ideas that came to my mind are below. **Option 1 (TableProperties)** We will have to add multiple new properties. For example, we might use something like this: ``` metrics.collection.enabled metrics.collection.column-delimiter metrics.collection.bounds.enabled metrics.collection.bounds.columns ``` Instead of storing a list of columns, we can store a regular expression. Then we will be able to construct a `MetricsSpec` based on those properties and pass it to writers. **Option 2 (Extend Metadata)** The previous option is a bit cumbersome. Alternatively, we can store a `MetricsSpec` directly in the metadata (similar to `PartitionSpec`) and expose an API to update it. Also, users will be able to provide a correct `MetricsSpec` while creating tables. @rdblue any suggestions on this?
---------------------------------------------------------------- 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]
