Github user twalthr commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4585#discussion_r138889015
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/calcite/FlinkTypeFactory.scala
 ---
    @@ -211,6 +218,14 @@ class FlinkTypeFactory(typeSystem: RelDataTypeSystem) 
extends JavaTypeFactoryImp
         canonize(relType)
       }
     
    +  override def createMultisetType(elementType: RelDataType, 
maxCardinality: Long): RelDataType = {
    +    val relType = new MultisetRelDataType(
    --- End diff --
    
    There are multiple location where a new type has to be added like 
`FlinkRelNode`.


---

Reply via email to