Re: [PR] [FLINK-34902][table] Fix IndexOutOfBoundsException for VALUES [flink]

2024-04-30 Thread via GitHub
twalthr merged PR #24724: URL: https://github.com/apache/flink/pull/24724 -- 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:

Re: [PR] [FLINK-34902][table] Fix IndexOutOfBoundsException for VALUES [flink]

2024-04-29 Thread via GitHub
jeyhunkarimov commented on PR #24724: URL: https://github.com/apache/flink/pull/24724#issuecomment-2082785290 Hi @twalthr thanks a lot for your review. Could you please recheck in your available time? Thanks -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [FLINK-34902][table] Fix IndexOutOfBoundsException for VALUES [flink]

2024-04-29 Thread via GitHub
twalthr commented on code in PR #24724: URL: https://github.com/apache/flink/pull/24724#discussion_r1582644328 ## flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/calcite/FlinkCalciteSqlValidatorTest.java: ## @@ -63,6 +64,55 @@ void testInsertInto2()

Re: [PR] [FLINK-34902][table] Fix IndexOutOfBoundsException for VALUES [flink]

2024-04-26 Thread via GitHub
reswqa commented on PR #24724: URL: https://github.com/apache/flink/pull/24724#issuecomment-2078864754 @twalthr, would you mind taking a look at this? thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [FLINK-34902][table] Fix IndexOutOfBoundsException for VALUES [flink]

2024-04-25 Thread via GitHub
flinkbot commented on PR #24724: URL: https://github.com/apache/flink/pull/24724#issuecomment-2078170997 ## CI report: * abf08e9eba3db5462b8634078f8ef2b1a3d3a165 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-34902][table] Fix IndexOutOfBoundsException for VALUES [flink]

2024-04-25 Thread via GitHub
jeyhunkarimov opened a new pull request, #24724: URL: https://github.com/apache/flink/pull/24724 ## What is the purpose of the change The purpose of this PR is to avoid wrong `IndexOutOfBoundsException` with Values. For example, a query `INSERT INTO t_3_columns (id, name, num)