[ 
https://issues.apache.org/jira/browse/FLINK-8038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16255277#comment-16255277
 ] 

ASF GitHub Bot commented on FLINK-8038:
---------------------------------------

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

    https://github.com/apache/flink/pull/5015#discussion_r151408733
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/validate/FunctionCatalog.scala
 ---
    @@ -333,6 +337,10 @@ class BasicOperatorTable extends 
ReflectiveSqlOperatorTable {
         SqlStdOperatorTable.ITEM,
         SqlStdOperatorTable.CARDINALITY,
         SqlStdOperatorTable.ELEMENT,
    +    // MAP OPERATORS
    +    SqlStdOperatorTable.MAP_VALUE_CONSTRUCTOR,
    +    SqlStdOperatorTable.ITEM,
    --- End diff --
    
    Btw there is no code that implements CARDINALITY. Can you add this call?


> Support MAP value constructor
> -----------------------------
>
>                 Key: FLINK-8038
>                 URL: https://issues.apache.org/jira/browse/FLINK-8038
>             Project: Flink
>          Issue Type: New Feature
>          Components: Table API & SQL
>            Reporter: Rong Rong
>            Assignee: Rong Rong
>
> Similar to https://issues.apache.org/jira/browse/FLINK-4554
> We want to support Map value constructor which is supported by Calcite:
> https://calcite.apache.org/docs/reference.html#value-constructors
> {code:sql}
> SELECT
>   MAP['key1', f0, 'key2', f1] AS stringKeyedMap,
>   MAP['key', 'value'] AS literalMap,
>   MAP[f0, f1] AS fieldMap
> FROM
>   table
> {code}
> This should enable users to construct MapTypeInfo, one of the CompositeType.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to