Github user sunjincheng121 commented on a diff in the pull request:
https://github.com/apache/flink/pull/5202#discussion_r160730995
--- Diff:
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/codegen/calls/FunctionGenerator.scala
---
@@ -414,6 +414,34 @@ object FunctionGenerator {
DOUBLE_TYPE_INFO,
BuiltInMethods.LOG_WITH_BASE)
+ addSqlFunctionMethod(
--- End diff --
Yes, that's correct..
+1 to add `shift` as a binary operator.---
