Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-10 Thread via GitHub
twalthr merged PR #26638: URL: https://github.com/apache/flink/pull/26638 -- 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.apach

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
twalthr commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132132754 ## flink-table/flink-table-common/src/main/java/org/apache/flink/table/api/DataTypes.java: ## @@ -928,37 +928,88 @@ public static UnresolvedDataType RAW(TypeInformati

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
twalthr commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132142649 ## flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/functions/StructuredFunctionsITCase.java: ## @@ -0,0 +1,120 @@ +/* + * Licensed to the A

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
twalthr commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132126772 ## flink-table/flink-sql-parser/src/main/java/org/apache/flink/table/calcite/ExtendedRelTypeFactory.java: ## @@ -33,4 +35,10 @@ public interface ExtendedRelTypeFactory

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
snuyanzin commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132125616 ## flink-table/flink-table-common/src/test/java/org/apache/flink/table/types/LogicalTypesTest.java: ## @@ -797,42 +831,46 @@ private StructuredType createHumanType(bo

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
snuyanzin commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132125616 ## flink-table/flink-table-common/src/test/java/org/apache/flink/table/types/LogicalTypesTest.java: ## @@ -797,42 +831,46 @@ private StructuredType createHumanType(bo

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
twalthr commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132107949 ## docs/content.zh/docs/dev/table/types.md: ## @@ -1301,35 +1349,27 @@ class User { // enrich the extraction with forcing using RAW types public @DataTypeHint

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
raminqaf commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132110036 ## docs/content.zh/docs/dev/table/types.md: ## @@ -1301,35 +1349,27 @@ class User { // enrich the extraction with forcing using RAW types public @DataTypeHin

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
twalthr commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132100150 ## docs/content.zh/docs/dev/table/types.md: ## @@ -1281,15 +1293,51 @@ For some classes an annotation is required in order to map the class to a data t to assign a fi

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
twalthr commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2132094866 ## docs/content.zh/docs/dev/table/types.md: ## @@ -1281,15 +1293,51 @@ For some classes an annotation is required in order to map the class to a data t to assign a fi

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-06 Thread via GitHub
raminqaf commented on code in PR #26638: URL: https://github.com/apache/flink/pull/26638#discussion_r2131553523 ## docs/content.zh/docs/dev/table/types.md: ## @@ -1281,15 +1293,51 @@ For some classes an annotation is required in order to map the class to a data t to assign a f

[PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-05 Thread via GitHub
twalthr opened a new pull request, #26638: URL: https://github.com/apache/flink/pull/26638 ## What is the purpose of the change *(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on

Re: [PR] [FLINK-37862][table] Support inline structured types in SQL [flink]

2025-06-05 Thread via GitHub
flinkbot commented on PR #26638: URL: https://github.com/apache/flink/pull/26638#issuecomment-2944347793 ## CI report: * 69434abb93f4968815acd6be039573508bc92085 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`