nastra commented on code in PR #13700: URL: https://github.com/apache/iceberg/pull/13700#discussion_r2247191137
########## arrow/src/main/java/org/apache/iceberg/arrow/vectorized/ArrowReader.java: ########## @@ -91,9 +92,8 @@ * <li>Columns with constant values are physically encoded as a dictionary. The Arrow vector type * is int32 instead of the type as per the schema. See * https://github.com/apache/iceberg/issues/2484. - * <li>Data types: {@link Types.ListType}, {@link Types.MapType}, {@link Types.StructType}, {@link - * Types.FixedType} and {@link Types.DecimalType} See - * https://github.com/apache/iceberg/issues/2485 and + * <li>Data types: {@link Types.ListType}, {@link Types.MapType}, {@link Types.StructType}, and + * {@link Types.DecimalType} See https://github.com/apache/iceberg/issues/2485 and Review Comment: decimal type support also exists already, so we can remove that comment -- 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]
