flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From twalthr <...@git.apache.org>
Subject [GitHub] flink pull request #4585: [FLINK-7491] [Table API & SQL] add MultiSetTypeInf...
Date Thu, 14 Sep 2017 13:18:08 GMT
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`.


---

Mime
View raw message