rdblue commented on code in PR #16961:
URL: https://github.com/apache/iceberg/pull/16961#discussion_r4127918540


##########
format/spec.md:
##########
@@ -825,6 +825,7 @@ Each stats struct holds statistics for one table field. It 
may contain the follo
 | _optional_  | 5      | `null_value_count`        | `long`                    
| optional fields                               | Number of null values in the 
column |
 | _optional_  | 6      | `nan_value_count`         | `long`                    
| `float`, `double`                             | Number of NaN values in the 
column |
 | _optional_  | 7      | `avg_value_size_in_bytes` | `int`                     
| `string`, `binary`, `variant`, `geometry`, `geography` | Avg value size in 
memory (uncompressed) in bytes over non-null values to estimate memory 
consumption |
+|             | 8      | `group_max_value`         | Field type                
| index cluster fields                          | Value of the field in the 
last row in [clustering order](index-spec.md#clustering-and-ordering); not 
necessarily the greatest value |

Review Comment:
   I don't understand the purpose of this. I would assume that no modifications 
to the table spec are needed for the index spec.



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

Reply via email to