huaxingao commented on code in PR #15180:
URL: https://github.com/apache/iceberg/pull/15180#discussion_r2796088414
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -4147,6 +4249,18 @@ components:
items:
$ref: '#/components/schemas/Namespace'
+ LoadFunctionResponse:
+ type: object
+ required:
+ - spec
Review Comment:
For Phase 1, I’m thinking of keeping this read-only (list/load) and focusing
on the minimal requirement: return the function spec document inline (spec). So
I’d prefer not adding metadata-location/metadata/config right now. When we add
CRUD, we can reconsider whether those table/view-style fields such as
metadata-location and metadata are needed for functions. WDYT?
--
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]
