rdblue commented on a change in pull request #1509:
URL: https://github.com/apache/iceberg/pull/1509#discussion_r495185377
##########
File path: flink/src/main/java/org/apache/iceberg/flink/TypeToFlinkType.java
##########
@@ -100,8 +100,8 @@ public LogicalType primitive(Type.PrimitiveType primitive) {
case DATE:
return new DateType();
case TIME:
- // MICROS
- return new TimeType(6);
+ // Flink only support TimeType with default precision now.
Review comment:
Can you update the comment to state what Flink's default precision is?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
