[GitHub] [flink] matriv commented on a change in pull request #19190: [FLINK-13785][table] Port time functions to new type inference

2022-03-31 Thread GitBox
matriv commented on a change in pull request #19190: URL: https://github.com/apache/flink/pull/19190#discussion_r839874307 ## File path: flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/CallContext.java ## @@ -93,6 +93,17 @@ default Validatio

[GitHub] [flink] matriv commented on a change in pull request #19190: [FLINK-13785][table] Port time functions to new type inference

2022-03-31 Thread GitBox
matriv commented on a change in pull request #19190: URL: https://github.com/apache/flink/pull/19190#discussion_r839868945 ## File path: flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/strategies/ToTimestampLtzTypeStrategy.java ## @@ -0,0 +1

[GitHub] [flink] matriv commented on a change in pull request #19190: [FLINK-13785][table] Port time functions to new type inference

2022-03-31 Thread GitBox
matriv commented on a change in pull request #19190: URL: https://github.com/apache/flink/pull/19190#discussion_r839867430 ## File path: flink-table/flink-table-common/src/main/java/org/apache/flink/table/functions/BuiltInFunctionDefinitions.java ## @@ -1222,76 +1231,103 @@

[GitHub] [flink] matriv commented on a change in pull request #19190: [FLINK-13785][table] Port time functions to new type inference

2022-03-30 Thread GitBox
matriv commented on a change in pull request #19190: URL: https://github.com/apache/flink/pull/19190#discussion_r838375382 ## File path: flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/strategies/ExtractInputTypeStrategy.java ## @@ -0,0 +1,1