flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kurt Young (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-12611) Make time indicator nullable in blink
Date Fri, 24 May 2019 07:57:00 GMT

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

Kurt Young commented on FLINK-12611:
------------------------------------

cc [~twalthr], what do you think about this case, it also relates how we define row time in
new type system.

> Make time indicator nullable in blink
> -------------------------------------
>
>                 Key: FLINK-12611
>                 URL: https://issues.apache.org/jira/browse/FLINK-12611
>             Project: Flink
>          Issue Type: Improvement
>          Components: Table SQL / Planner
>            Reporter: Jingsong Lee
>            Assignee: Jingsong Lee
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> SQL: select max(rowtime), count(a) from T
> There will be a AssertionError: type mismatch:
> aggCall type:
> TIMESTAMP(3) NOT NULL
> inferred type:
> TIMESTAMP(3)
> Agg type checking is done before TimeIndicator materializes. So there is a exception.
> And before introducing nullable of LogicalType, we should modify this to avoid more potential
TypeCheck problems.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message