flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "sunjincheng (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-6737) Fix over expression parse String error.
Date Mon, 29 May 2017 06:29:04 GMT

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

sunjincheng commented on FLINK-6737:

Hi [~fhueske]I think in this JIRA. only improve `over` method, not whole `select` clause.
I have not met the case that is not supported by a Scala expression but only by a String expression.

I think this is not a release blocker. If you do not want add this change, I'am okay.


> Fix over expression parse String error.
> ---------------------------------------
>                 Key: FLINK-6737
>                 URL: https://issues.apache.org/jira/browse/FLINK-6737
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table API & SQL
>    Affects Versions: 1.3.0
>            Reporter: sunjincheng
>            Assignee: sunjincheng
> When we run the TableAPI as follows:
> {code}
> val windowedTable = table
>       .window(Over partitionBy 'c orderBy 'proctime preceding UNBOUNDED_ROW as 'w)
>       .select('c, "countFun(b)" over 'w as 'mycount, weightAvgFun('a, 'b) over 'w as
> {code}
> We get the error:
> {code}
> org.apache.flink.table.api.TableException: The over method can only using with aggregation
> 	at org.apache.flink.table.api.scala.ImplicitExpressionOperations$class.over(expressionDsl.scala:469)
> 	at org.apache.flink.table.api.scala.ImplicitExpressionConversions$LiteralStringExpression.over(expressionDsl.scala:756)
> {code}
> The reason is, the `over` method of `expressionDsl` not parse the String case.
> I think we should fix this before 1.3 release.

This message was sent by Atlassian JIRA

View raw message