szehon-ho commented on code in PR #17855:
URL: https://github.com/apache/iceberg/pull/17855#discussion_r3919882601
##########
open-api/rest-catalog-open-api.py:
##########
@@ -1101,6 +1109,11 @@ class ContentFile(BaseModel):
None, alias='split-offsets', description='List of splittable offsets'
)
sort_order_id: int | None = Field(None, alias='sort-order-id')
+ content_stats: dict[str, FieldStatistics] | None = Field(
+ None,
+ alias='content-stats',
+ description='Container struct for per-field metrics structs',
Review Comment:
Could we document that `content-stats` keys are decimal string field IDs,
ideally with an example? The parser requires numeric keys, while the OpenAPI
schema currently allows arbitrary strings. Fine to address in a follow-up.
--
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]
