alamb commented on code in PR #7451:
URL: https://github.com/apache/arrow-rs/pull/7451#discussion_r2080226794


##########
arrow-avro/src/codec.rs:
##########
@@ -162,6 +175,8 @@ pub enum Codec {
     List(Arc<AvroDataType>),
     /// Represents Avro record type, maps to Arrow's Struct data type
     Struct(Arc<[AvroField]>),
+    /// Represents Avro map type, maps to Arrow's Map data type

Review Comment:
   Since this is a `pub` enum and it is not marked as `non exhaustive` I think 
this is technically a breaking API change and will need to wait for the next 
major release:
   - #7395 



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to