[
https://issues.apache.org/jira/browse/FLINK-12611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16847349#comment-16847349
]
Kurt Young commented on FLINK-12611:
------------------------------------
[~jark] This will work, the query will have a result filed with type: BIGINT
NULLABLE
The problem with the original query is: does the result of max(rowtime) still a
valid rowtime? If yes, it will violate the constraint that the rowtime should
always be not null. If not, then we will loose the rowtime attribute after
aggregate it.
> 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)
