Github user twalthr commented on a diff in the pull request:
https://github.com/apache/flink/pull/4127#discussion_r160629614
--- Diff:
flink-libraries/flink-table/src/test/scala/org/apache/flink/table/expressions/ScalarFunctionsTest.scala
---
@@ -352,6 +352,24 @@ class ScalarFunctionsTest extends ScalarTypesTestBase {
"Flink~~~~xx")
}
+ @Test
+ def testLpad(): Unit = {
+ testSqlApi("LPAD('hi',4,'??')", "??hi")
+ testSqlApi("LPAD('hi',1,'??')", "h")
+ testSqlApi("LPAD('',1,'??')", "?")
+ testSqlApi("LPAD('',30,'??')", "??????????????????????????????")
+ testSqlApi("LPAD('111',-2,'??')", "")
--- End diff --
Is `testSqlApi("LPAD('111',-2,'??')", "")` standard?
I just tested this with MySQL and it returns NULL
---
