jackye1995 commented on code in PR #9292:
URL: https://github.com/apache/iceberg/pull/9292#discussion_r1425987620
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -2367,6 +2368,21 @@ components:
items:
type: string
+ AppendFilesUpdate:
+ allOf:
+ - $ref: '#/components/schemas/BaseUpdate'
+ required:
+ - appended-manifests
+ properties:
+ action:
+ type: string
+ enum: [ "append-files" ]
+ appended-manifests:
Review Comment:
I think we did not really finalize in the discussion threads if this is a
good idea. I think it's an easy integration with existing JVM-based engines
since they already know how to produce manifests, but if our goal is to make it
easy for other non-JVM engines to integrate, seems like this should be a list
of DataFiles, or data file locations.
--
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
