RussellSpitzer commented on code in PR #14888: URL: https://github.com/apache/iceberg/pull/14888#discussion_r2662213382
########## docs/docs/api.md: ########## @@ -188,6 +188,47 @@ MapType map = MapType.ofOptional( ListType list = ListType.ofRequired(1, IntegerType.get()); ``` +## Using Version 3 Types Review Comment: @evindj Can we please remove these doc changes? I just don't think they add very much here, "nanosecond" is just another primitive that we've already gone over. The Variant and Geospatial examples don't really show much either. I think we are good with just updating the other page -- 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]
