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

Reply via email to