[GitHub] [flink] HuangXingBo commented on a change in pull request #9969: [FLINK-14022][table-planner][table-planner-blink] Add validation check for places where Python ScalarFunction cannot be used

[GitBox <gi...@apache.org>]

HuangXingBo commented on a change in pull request #9969: [FLINK-14022][table-planner][table-planner-blink] Add validation check for places where Python ScalarFunction cannot be used
URL: https://github.com/apache/flink/pull/9969#discussion_r341478472
 
 

 ##########
 File path: flink-table/flink-table-planner/src/test/scala/org/apache/flink/table/api/stream/sql/validation/MatchRecognizeValidationTest.scala
 ##########
 @@ -42,4 +44,25 @@ class MatchRecognizeValidationTest extends TableTestBase {
     val sql = "SELECT MATCH_PROCTIME() FROM MyTable"
     streamUtil.verifySql(sql, "n/a")
   }
+
+  /** Python Function can not be used in MATCH_RECOGNIZE for now **/
+  @Test
+  def testMatchPythonFunction() = {
+    expectedException.expectCause(Matchers.isA(classOf[TableException]))
 
 Review comment:
   Because in flink planner, this kind of exceptions thrown in physicalNode initiailization is wrapped in a new RuntimeException with new error message, it is hard to catch the error message what we want.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services