twalthr commented on a change in pull request #18858:
URL: https://github.com/apache/flink/pull/18858#discussion_r812624863



##########
File path: 
flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/planner/functions/utils/ScalarSqlFunction.scala
##########
@@ -42,7 +44,10 @@ import scala.collection.JavaConverters._
   * @param displayName    name to be displayed in operator name
   * @param scalarFunction scalar function to be called
   * @param typeFactory    type factory for converting Flink's between 
Calcite's types
+  * @deprecated Use [[BuiltInFunctionDefinitions]] that translates to 
[[BridgingSqlFunction]].
   */
+@Deprecated
+@deprecated

Review comment:
       yes, one for Scala and one for Java




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to