[FLINK-6737] Fix over expression parse String error.

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.3.0
• Fix Version/s: 1.3.0, 1.4.0
• Component/s: Table SQL / API
• Assignee: sunjincheng
• Reporter: sunjincheng
• Created: 27/May/17 04:44
• Updated: 29/May/17 14:18
• Resolved: 29/May/17 14:18

Description

When we run the TableAPI as follows:

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 'wAvg)

We get the error:

org.apache.flink.table.api.TableException: The over method can only using with aggregation expression.

	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)

The reason is, the `over` method of `expressionDsl` not parse the String case.
I think we should fix this before 1.3 release.

Issue Links

• links to: GitHub Pull Request #4000