rdblue commented on code in PR #12584: URL: https://github.com/apache/iceberg/pull/12584#discussion_r2198654847
########## 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: Review Comment: Thinking through how catalogs will evolve to support transactions, I think it is likely that some of them will have a catalog-level sequence number for changes. This schema doesn't prevent us from either allowing that in a response or adding it in the future, right? I don't think so but I want to raise it for everyone to think about. -- 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