stevenzwu commented on code in PR #15830:
URL: https://github.com/apache/iceberg/pull/15830#discussion_r3120356913
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3787,6 +3941,164 @@ components:
additionalProperties:
type: string
+ CatalogObjectType:
+ type: string
+ description: |
+ The type of a catalog object.
+ enum:
Review Comment:
the difference is the response object is different for MV that contains both
view metadata and storage table metadata. If we want to load a MV in one round
trip, the response payload would compose of both view and storage table. A
specific MV type is needed so that client knows how to parse the response.
In the Java library, we might need to also define a MaterializedView type,
which could be mostly just a container class for a `View` and a `Table` fields.
--
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]
