[GitHub] [iceberg] kbendick removed a comment on pull request #3770: Spec: Initial OpenAPI Spec for a Rest Catalog

[GitBox <gi...@apache.org>]

kbendick removed a comment on pull request #3770:
URL: https://github.com/apache/iceberg/pull/3770#issuecomment-997120898


   > Overall, this looks good to me. I think that one of the main things to do is add consistency since there are a lot of differences that make it hard to consume as the yaml spec. I think that part of the motivation is to not have a huge number of schemas and response types so this is embedding them in some of the routes. But that ends up being a little confusing overall.
   
   Yeah. With the OpenAPI doc, there really is either a choice between a long form document (where everything is spelled out), or a shorter form document that is harder to read if you're not used to looking at OpenAPI docs.
   
   I'll make it more consistent, but I will warn you that the document size will likely double.


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org