Github user twalthr commented on a diff in the pull request:
https://github.com/apache/flink/pull/6027#discussion_r189895031
--- Diff:
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/expressions/ExpressionParser.scala
---
@@ -210,6 +212,14 @@ object ExpressionParser extends JavaTokenParsers with
PackratParsers {
case agg ~ _ ~ windowRef => UnresolvedOverCall(agg, windowRef)
}
+ lazy val between: PackratParser[Expression] =
+ expression ~ "." ~ (BETWEEN | NOT_BETWEEN) ~ "(" ~ expression ~ "," ~
expression ~ ")" ^^ {
--- End diff --
The changes in `ExpressionParser` should not be necessary if you add the
expression classes to `org.apache.flink.table.validate.FunctionCatalog`. What
happens if you remove these changes?
---
