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/27 07:46:15 UTC

[GitHub] [iceberg] snazy commented on pull request #6672: Add REST catalog spec to iceberg-core.jar

snazy commented on PR #6672:
URL: https://github.com/apache/iceberg/pull/6672#issuecomment-1406133195

   I think it's useful for two purposes:
   1. to have the spec together with the code, to know how the spec for the code look(ed|s) like
   2. to be able to generate some code - at least as a starter
   
   Types are "an issue", yes. Not sure how much it would affect generated code in general  / across all languages. Jackson for example handles a "generic `JsonNode`" (which can be "anything") perfectly fine - I imagine that it would work similar for a "generic `j.l.Object`".


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