Fokko commented on code in PR #9870: URL: https://github.com/apache/iceberg/pull/9870#discussion_r1559451439
########## 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: > So, As of now (since spec is based on v3), it is a required field. So, lets update the description and merge this as it is confusing. I'm also in favor of leaving it like this. It is just yet another thing we cannot express in the open-API specification, but the intent is [that the prefix is optional](https://github.com/apache/iceberg-python/blob/5039b5d70644bc06c98349090912c6e9066d3ea1/pyiceberg/catalog/rest.py#L285). -- 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