kbendick commented 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: [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]
