rdblue commented on code in PR #6214: URL: https://github.com/apache/iceberg/pull/6214#discussion_r1033021155
########## format/view-spec.md: ########## @@ -32,14 +32,11 @@ Each compute engine stores the metadata of the view in its proprietary format in ## Goals * A common metadata format for view metadata, similar to how Iceberg supports a common table format for tables. -* The view metadata format specification - * Includes storage format as well as APIs to write/read the metadata. - * Supports versioning of views to track how a view evolved over time. ## Overview View metadata storage mirrors how Iceberg table metadata is stored and retrieved. View metadata is maintained in metadata files. All changes to view state create a new view metadata file and completely replace the old metadata using an atomic swap. Like Iceberg tables, this atomic swap is delegated to the metastore that tracks tables and/or views by name. The view metadata file tracks the view schema, custom properties, current and past versions, as well as other metadata. -Each metadata file is self-sufficient. It contains the history of the last few operations performed on the view and can be used to roll back the view to a previous version. +Each metadata file is self-sufficient. It contains the history of the last few operations performed on the view. The view can be roll back to a previous version using these version ids. Review Comment: I don't think this needs to change. -- 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]
