You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "snazy (via GitHub)" <gi...@apache.org> on 2023/01/30 09:43:30 UTC

[GitHub] [iceberg] snazy opened a new pull request, #6699: OpenAPI responses should reference schemas

snazy opened a new pull request, #6699:
URL: https://github.com/apache/iceberg/pull/6699

   The common _OpenAPI Tools_ generators do not properly recognize non-200 responses and generate the necessary objects for the non-200 response types. This change moves the schema definitions from `responses` to `schemas`.
   
   Having code generated from the spec helps a lot, at least as a good start.


-- 
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


[GitHub] [iceberg] rdblue merged pull request #6699: OpenAPI responses should reference schemas

Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue merged PR #6699:
URL: https://github.com/apache/iceberg/pull/6699


-- 
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


[GitHub] [iceberg] rdblue commented on pull request #6699: OpenAPI responses should reference schemas

Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on PR #6699:
URL: https://github.com/apache/iceberg/pull/6699#issuecomment-1493407286

   @snazy, this is just moving the types around and doesn't introduce any other changes, right?


-- 
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


[GitHub] [iceberg] snazy commented on pull request #6699: OpenAPI responses should reference schemas

Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on PR #6699:
URL: https://github.com/apache/iceberg/pull/6699#issuecomment-1506916799

   > this is just moving the types around and doesn't introduce any other changes, right?
   
   correct


-- 
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