You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "MartijnVisser (via GitHub)" <gi...@apache.org> on 2026/09/23 07:00:46 UTC

[PR] [BP-4.1][FLINK-40725][tests] Fix sink/catalog ITCases for Flink 2.3.0 ON CONFLICT validation [flink-connector-jdbc]

MartijnVisser opened a new pull request, #249:
URL: https://github.com/apache/flink-connector-jdbc/pull/249

   Backport of https://github.com/apache/flink-connector-jdbc/pull/232 (9581669), which went to `main` but not to `v4.1`.
   
   Flink 2.3 rejects an upsert whose key differs from the sink's primary key unless the `INSERT` has an `ON CONFLICT` clause. The sink and catalog ITCases add `ON CONFLICT DO DEDUPLICATE` on 2.3 and later, and push CI gains a 2.3.0 leg.
   
   Last occurrence on `v4.1`: https://github.com/apache/flink-connector-jdbc/actions/runs/35779813059/job/106922160499 (`compile_and_test (2.3.0, v4.1) / compile_and_test (17)`). Locally, `DerbyDynamicTableSinkITCase` fails on `v4.1` against 2.3.0 without this and passes with it on 2.3.0 and 2.2.1; `MySqlCatalogITCase` and `OceanBaseMysqlCatalogITCase` pass on 2.3.0.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes (please specify the tool below)
   
   Generated-by: Claude Code (Claude Opus 5.5)
   


-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [BP-4.1][FLINK-40725][tests] Fix sink/catalog ITCases for Flink 2.3.0 ON CONFLICT validation [flink-connector-jdbc]

Posted by "MartijnVisser (via GitHub)" <gi...@apache.org>.
MartijnVisser commented on PR #249:
URL: https://github.com/apache/flink-connector-jdbc/pull/249#issuecomment-5790531051

   @spuru9 Do you want to take a look?


-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [BP-4.1][FLINK-40725][tests] Fix sink/catalog ITCases for Flink 2.3.0 ON CONFLICT validation [flink-connector-jdbc]

Posted by "MartijnVisser (via GitHub)" <gi...@apache.org>.
MartijnVisser merged PR #249:
URL: https://github.com/apache/flink-connector-jdbc/pull/249


-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org