danny0405 commented on a change in pull request #14451:
URL: https://github.com/apache/flink/pull/14451#discussion_r549275650
##########
File path:
flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/plan/schema/MapRelDataType.scala
##########
@@ -45,5 +45,4 @@ class MapRelDataType(
override def hashCode(): Int = {
Objects.hashCode(keyType, valueType)
}
-
Review comment:
Yes, the map type has nullability issue for its nested key and values,
because currently we create the map type and ignore its key/value nullability
(both of them are from the type information with nullability true as default).
----------------------------------------------------------------
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:
us...@infra.apache.org
