Github user fhueske commented on a diff in the pull request:
https://github.com/apache/flink/pull/5638#discussion_r182104673
--- Diff:
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/api/scala/expressionDsl.scala
---
@@ -1130,4 +1130,13 @@ object concat_ws {
}
}
+object log {
+ def apply(base: Expression, antilogarithm: Expression): Expression = {
+ Log(base, antilogarithm)
+ }
+ def apply(antilogarithm: Expression): Expression = {
+ new Log(antilogarithm)
--- End diff --
We should go for `antilog.log(base)` for consistency with `antilog.ln()`
and `antilog.log10()`.
Since we need to inverse the parameters of the case class, we need to adapt
the `ExpressionParser`.
---
