[GitHub] [flink] lincoln-lil commented on a diff in pull request #23075: [FLINK-32674][doc] Add documentation for the new getTargetColumns in DynamicTableSink

2023-07-26 Thread via GitHub
lincoln-lil commented on code in PR #23075: URL: https://github.com/apache/flink/pull/23075#discussion_r1274924840 ## docs/content/docs/dev/table/sql/insert.md: ## @@ -208,7 +208,9 @@ column_list: **COLUMN LIST** Given a table T(a INT, b INT, c INT), Flink supports INSERT

[GitHub] [flink] lincoln-lil commented on a diff in pull request #23075: [FLINK-32674][doc] Add documentation for the new getTargetColumns in DynamicTableSink

2023-07-26 Thread via GitHub
lincoln-lil commented on code in PR #23075: URL: https://github.com/apache/flink/pull/23075#discussion_r1274924840 ## docs/content/docs/dev/table/sql/insert.md: ## @@ -208,7 +208,9 @@ column_list: **COLUMN LIST** Given a table T(a INT, b INT, c INT), Flink supports INSERT

[GitHub] [flink] lincoln-lil commented on a diff in pull request #23075: [FLINK-32674][doc] Add documentation for the new getTargetColumns in DynamicTableSink

2023-07-26 Thread via GitHub
lincoln-lil commented on code in PR #23075: URL: https://github.com/apache/flink/pull/23075#discussion_r1274930002 ## docs/content/docs/dev/table/sql/insert.md: ## @@ -208,7 +208,9 @@ column_list: **COLUMN LIST** Given a table T(a INT, b INT, c INT), Flink supports INSERT