[GitHub] [iceberg] aokolnychyi commented on pull request #5376: Core: Add readable metrics columns to files metadata tables

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

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: 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