Fokko commented on code in PR #13181: URL: https://github.com/apache/iceberg/pull/13181#discussion_r2266851932
########## format/spec.md: ########## @@ -814,8 +814,8 @@ Manifest list files store `manifest_file`, a struct with the following fields: `field_summary` is a struct with the following fields: -| v1 | v2 | Field id, name | Type | Description | -| ---------- | ---------- |-------------------------|---------------|-------------| +| v1 | v2+ | Field id, name | Type | Description | +| ---------- |------------|-------------------------|---------------|-------------| Review Comment: Nit: We seem to mix the `| --- |` and `|-----|` style. GitHub Flavoured Markdown seems to suggest the first: https://github.github.com/gfm/#tables-extension- -- 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]
