jackye1995 commented on code in PR #6861:
URL: https://github.com/apache/iceberg/pull/6861#discussion_r1129749738
##########
api/src/main/java/org/apache/iceberg/view/ViewVersion.java:
##########
@@ -43,8 +47,7 @@ public interface ViewVersion {
long timestampMillis();
/**
- * Return the version summary such as the name of the operation that created
that version of the
- * view
+ * Return a string map of summary data for the operation that produced this
view version.
Review Comment:
I think the argument also applies to the data model, because for Snapshot I
am interested in what is the operation that produces the snapshot, because
there are different types for data write. But for view, I am not sure how
useful that information is, it's always just create or replace of the metadata
definition. So I don't know if we should add this layer of complication.
--
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]
