jackye1995 commented on code in PR #9717: URL: https://github.com/apache/iceberg/pull/9717#discussion_r1504590935
########## open-api/rest-catalog-open-api.yaml: ########## @@ -3324,6 +3348,184 @@ components: type: integer format: int64 + BooleanTypeValue: + type: boolean + + IntegerTypeValue: + type: integer + + LongTypeValue: + type: integer + format: int64 + + FloatTypeValue: Review Comment: Sure, we can move forward with this first, although I am not fully convinced yet, but I don't want to block on this for too long. I think the issues I see are mostly due to the serialization implementation of Jackson that does not comply with the standards you listed above. We can address that in subsequent implementation PRs. -- 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