Fokko commented on code in PR #9870:
URL: https://github.com/apache/iceberg/pull/9870#discussion_r1559231592
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -1444,7 +1444,7 @@ components:
schema:
type: string
required: true
- description: An optional prefix in the path
+ description: Prefix in the path
Review Comment:
Optional path parameters are not allowed in the V3 spec:
https://swagger.io/docs/specification/describing-parameters/ So the validator
is right.
Let's not duplicate the paths since that's going to be confusing as JB
pointed out. There is a long discussion going on here:
https://github.com/OAI/OpenAPI-Specification/issues/93#issuecomment-1908673473
And it looks like it gets support in V4. I would say that we wait until that's
being released and solve it then.
--
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]
