rdblue commented on code in PR #11839:
URL: https://github.com/apache/iceberg/pull/11839#discussion_r1925882243


##########
flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/data/RowDataUtil.java:
##########
@@ -63,6 +65,27 @@ public static Object convertConstant(Type type, Object 
value) {
         return ByteBuffers.toByteArray((ByteBuffer) value);
       case BINARY: // byte[]
         return ByteBuffers.toByteArray((ByteBuffer) value);
+      case STRUCT: // struct

Review Comment:
   What is this used for? Right now we don't have struct default values so I 
don't think this will be exercised.



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