JanKaul commented on code in PR #11041: URL: https://github.com/apache/iceberg/pull/11041#discussion_r4144104269
########## format/view-spec.md: ########## @@ -63,11 +75,13 @@ The view version metadata file has the following fields: | _required_ | `versions` | A list of known [versions](#versions) of the view [1] | | _required_ | `version-log` | A list of [version log](#version-log) entries with the timestamp and `version-id` for every change to `current-version-id` | | _optional_ | `properties` | A string to string map of view properties [2] | +| _optional_ | `max-staleness-ms` | The maximum time interval in milliseconds during which changed source table snapshots are considered fresh enough to skip refreshing [3] | Review Comment: We removed "max-staleness" from the current spec as we couldn't agree on a common definition across all engines. That said, every engine can still apply it's own staleness window it's just not standardized across engines and it has to define it's own properties for it. Because it's such a common case we still list it as an example in the current version. We encourage it but we don't introduce a common definition that has to be enforced by every engine. -- 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]
