jacobmarble commented on code in PR #8683:
URL: https://github.com/apache/iceberg/pull/8683#discussion_r1360930771


##########
format/spec.md:
##########
@@ -874,6 +878,11 @@ Maps with non-string keys must use an array representation 
with the `map` logica
 |**`list`**|`array`||
 |**`map`**|`array` of key-value records, or `map` when keys are strings 
(optional).|Array storage must use logical type name `map` and must store 
elements that are 2-field records. The first field is a non-null key and the 
second field is the value.|
 
+Notes:
+
+1. Avro type annotation `adjust-to-utc` is an Iceberg convention; default 
value is `false` if not present.
+2. Avro logical type `timestamp-nanos` is an Iceberg convention; the Avro 
specification does not define this type.

Review Comment:
   @Fokko please drop a link to Avro devlist post when you have a moment. The 
mailing list link on [the Avro community 
page](https://avro.apache.org/community/) is just an email address, can't find 
the web UI.



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

Reply via email to