twalthr commented on a change in pull request #11568:
URL: https://github.com/apache/flink/pull/11568#discussion_r411178550
##########
File path:
flink-table/flink-table-planner-blink/src/main/scala/org/apache/flink/table/planner/calcite/FlinkTypeFactory.scala
##########
@@ -484,6 +493,9 @@ object FlinkTypeFactory {
// CURSOR for UDTF case, whose type info will never be used, just a
placeholder
case CURSOR => new TypeInformationRawType[Nothing](new NothingTypeInfo)
+ case null if relDataType.isInstanceOf[RawRelDataType] =>
Review comment:
@danny0405 thanks for the hint around `SqlTypeName.OTHER`. This one fits
better to the semantics of the RAW type than `SqlTypeName.ANY`. The family `
SqlTypeFamily.ANY` includes all types which is not correct for the RAW type.
But `SqlTypeName.OTHER` has no type family which is correct. I'm fine with
using `SqlTypeName.OTHER`.
----------------------------------------------------------------
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
