nastra commented on code in PR #8608:
URL: https://github.com/apache/iceberg/pull/8608#discussion_r1332843710
##########
core/src/main/java/org/apache/iceberg/view/ViewMetadata.java:
##########
@@ -375,6 +386,13 @@ public ViewMetadata build() {
Preconditions.checkArgument(null != location, "Invalid location: null");
Preconditions.checkArgument(versions.size() > 0, "Invalid view: no
versions were added");
+ // when associated with a metadata file, metadata must have no changes
so that the metadata
+ // matches exactly what is in the metadata file, which does not store
changes. metadata
+ // location with changes is inconsistent.
+ Preconditions.checkArgument(
Review Comment:
we don't want to fail in the `check()` method but rather in `build()`. Error
messaging is aligned with `TableMetadata`
--
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]
