c-thiel commented on code in PR #12584:
URL: https://github.com/apache/iceberg/pull/12584#discussion_r2112105575
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3041,6 +3096,34 @@ components:
- $ref: '#/components/schemas/AddViewVersionUpdate'
- $ref: '#/components/schemas/SetCurrentViewVersionUpdate'
+ OperationType:
+ type: string
+ description: >
+ Defines the type of operation, either a standard operation defined by
the Iceberg REST API
+ specification or a custom operation specific to an implementation.
+ anyOf:
+ - enum:
+ - create-table
Review Comment:
The matrix approach that you propose can lead to invalid combinations,
already today, register wouldn't be easy to distinguish from create. Also
"replace" is something that is only valid for views and not namespaces for
example. Because of this, I would prefer to stick to the current long list.
--
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
