c-thiel commented on code in PR #12584: URL: https://github.com/apache/iceberg/pull/12584#discussion_r2206717863
########## open-api/rest-catalog-open-api.yaml: ########## @@ -3964,6 +4144,292 @@ components: metadata: $ref: '#/components/schemas/TableMetadata' + EventsResponse: + type: object + required: + - highest-processed-timestamp-ms + - events + properties: + next-page-token: + $ref: "#/components/schemas/PageToken" + highest-processed-timestamp-ms: + description: > + The highest event timestamp processed when generating this response. + This may not necessarily appear in the returned changes if it was filtered out. + type: integer + format: int64 + events: + type: array + items: + $ref: "#/components/schemas/Event" + + Event: + type: object + required: + - event-id + - request-id + - event-count + - timestamp-ms + - operation + properties: + event-id: + type: string + description: Unique ID of this event. Clients should perform deduplication based on this ID. + request-id: + description: > + Opaque ID of the request this change belongs to. + This ID can be used to identify events that were part of the same request. + Servers generate this ID randomly. + type: string + event-count: + type: integer + description: Number of events in the request / batch of events + timestamp-ms: + type: integer + format: int64 + description: > + Timestamp when this transaction occurred (epoch milliseconds). + Timestamps are not guaranteed to be unique. Typically all events in + a transaction will have the same timestamp. + actor: + type: string + description: > + The actor who performed the operation, such as a user or service account. + The content of this field is implementation specific. + operation: + type: object + description: > + The operation that was performed, such as creating or updating a table. + Clients should discard events with unknown operation types. + discriminator: + propertyName: operation-type + mapping: + create-table: "#/components/schemas/CreateTableOperation" + register-table: "#/components/schemas/RegisterTableOperation" + drop-table: "#/components/schemas/DropTableOperation" + update-table: "#/components/schemas/UpdateTableOperation" + rename-table: "#/components/schemas/RenameTableOperation" + create-view: "#/components/schemas/CreateViewOperation" + drop-view: "#/components/schemas/DropViewOperation" + update-view: "#/components/schemas/UpdateViewOperation" + rename-view: "#/components/schemas/RenameTableOperation" + create-namespace: "#/components/schemas/CreateNamespaceOperation" + update-namespace-properties: "#/components/schemas/UpdateNamespacePropertiesOperation" + drop-namespace: "#/components/schemas/DropNamespaceOperation" + custom: "#/components/schemas/CustomOperation" + oneOf: + - $ref: "#/components/schemas/CreateTableOperation" + - $ref: "#/components/schemas/RegisterTableOperation" + - $ref: "#/components/schemas/DropTableOperation" + - $ref: "#/components/schemas/UpdateTableOperation" + - $ref: "#/components/schemas/RenameTableOperation" + - $ref: "#/components/schemas/CreateViewOperation" + - $ref: "#/components/schemas/DropViewOperation" + - $ref: "#/components/schemas/UpdateViewOperation" + - $ref: "#/components/schemas/RenameTableOperation" + - $ref: "#/components/schemas/CreateNamespaceOperation" + - $ref: "#/components/schemas/UpdateNamespacePropertiesOperation" + - $ref: "#/components/schemas/DropNamespaceOperation" + - $ref: "#/components/schemas/CustomOperation" + + CreateTableOperation: + required: + - operation-type + - identifier + - table-uuid + properties: Review Comment: I am not sure if the semantic of an `Update` makes sense if we don't have a proper initial state. The `CreateTableRequest` also works without updates. I believe that for a staged create, we also don't have a good way in our SDKs to obtain all updates, unless we cache all changes for the stage create and append updates of the commit. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org