ajantha-bhat commented on code in PR #6214: URL: https://github.com/apache/iceberg/pull/6214#discussion_r1026144988
########## format/view-spec.md: ########## @@ -58,16 +55,16 @@ Writers create view metadata files optimistically, assuming that the current met The view version metadata file has the following fields: -| Required/Optional | Field Name | Description | -|-------------------|------------|-------------| -| Required | format-version | An integer version number for the view format. Currently, this must be 1. Implementations must throw an exception if the view's version is higher than the supported version. | -| Required | location | The view's base location. This is used to determine where to store manifest files and view metadata files. | Review Comment: I was reading the spec today and these 'manifest files' caught my attention. Then I wanted to see why it is missed in the review. Later found that there are open comments on the same. So, handled most of the comments. -- 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]
