yiyutian1 commented on code in PR #25763:
URL: https://github.com/apache/flink/pull/25763#discussion_r1886086309
##########
flink-table/flink-table-planner/src/test/scala/org/apache/flink/table/planner/expressions/TemporalTypesTest.scala:
##########
@@ -1308,29 +1303,42 @@ class TemporalTypesTest extends ExpressionTestBase {
// invalid type for the first input
testExpectedSqlException(
"TO_TIMESTAMP_LTZ('test_string_type', 0)",
- "Cannot apply 'TO_TIMESTAMP_LTZ' to arguments of type" +
- " 'TO_TIMESTAMP_LTZ(<CHAR(16)>, <INTEGER>)'. Supported form(s):" +
- " 'TO_TIMESTAMP_LTZ(<NUMERIC>, <INTEGER>)'",
+ "SQL validation failed. From line 1, column 8 to line 1, column 46:
Cannot apply " +
Review Comment:
Need to change these existing scala tests, because I changed the function
signature. The change reflects the expected function behaviors.
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
