aokolnychyi commented on PR #5376: URL: https://github.com/apache/iceberg/pull/5376#issuecomment-1210148871
It seems like a great idea to add readable metrics. It is hard to make sense of them otherwise. @szehon-ho, what do you think about adding a single map column, let's say called `readable_metrics`, that will hold a mapping from a column name into a struct that would represent metrics? The type will be `Map<String, StructType>` and we will have individual struct fields for each type of metric. We can then easily access them via SQL. ``` SELECT readable_metrics['col1'].lower_bound FROM db.t.files ``` I am okay with individual columns too but it seems a bit cleaner to just have one. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
