wmoustafa commented on code in PR #9830:
URL: https://github.com/apache/iceberg/pull/9830#discussion_r1523520634
##########
core/src/main/java/org/apache/iceberg/view/ViewVersionReplace.java:
##########
@@ -56,6 +57,14 @@ public ViewVersion apply() {
}
ViewMetadata internalApply() {
+ // Replacing a materialized view is not supported because the old storage
location will wrongly
+ // transfer to the new version
+ // if not handled properly.
Review Comment:
Since this relates to the spec, I have decided to add this support. Now
there is a new view version property `materialized.view.version.id` that is
tracked at the table level and it is factored into freshness evaluation.
--
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]
