You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "GreatEugenius (via GitHub)" <gi...@apache.org> on 2026/06/24 05:49:32 UTC
[PR] [dist] Add Flink 2.3 distribution and CI coverage [flink-agents]
GreatEugenius opened a new pull request, #856:
URL: https://github.com/apache/flink-agents/pull/856
<!--
* Thank you very much for contributing to Flink Agents.
* Please add the relevant components in the PR title. E.g., [api], [runtime], [java], [python], [hotfix], etc.
-->
<!-- Please link the PR to the relevant issue(s). Hotfix doesn't need this. -->
Linked issue: #833
Purpose of change
Add Flink 2.3 as a supported distribution while keeping the build baseline at Flink 2.2.1 / Java 11.
Flink 2.3 jars are Java 17 bytecode, so 2.3 stays a provided runtime target only — core modules still compile once against the 2.2.1 baseline on JDK 11, mirroring how the MCP module isolates its 17-only dependency. This avoids forcing JDK 17 on every build and preserves backward-binary-compat across all supported Flink versions.
- dist: new flink-2.3 module with provided deps + thin-jar packaging
- e2e: flink-2.3 profile so IT runs the 2.2.1-built jar on a 2.3 runtime
- ci: add {flink 2.3, java 17} to the it-java and it-python matrices
- python: bundle the 2.3 thin jar into the wheel manifest, cover it in the build-backend test
- misc: refresh ut.sh help text and the cross-language async version notes
Tests
- Full multi-module build on JDK 11 (all 5 dist versions: 1.20/2.0/2.1/2.2/2.3)
- Java IT against Flink 2.3 runtime — 27 pas
- Python IT against Flink 2.3 runtime — 4 passed
- CI matrices extended with {flink 2.3, javapython
API
No public API changes. 2.3 is a packaging/ru
### Documentation
<!-- Do not remove this section. Check the proper box only. -->
- [ ] `doc-needed` <!-- Your PR changes impact docs -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [x] `doc-included` <!-- Your PR already contains the necessary documentation updates -->
--
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] [dist] Add Flink 2.3 distribution and make it the default [flink-agents]
Posted by "weiqingy (via GitHub)" <gi...@apache.org>.
weiqingy commented on code in PR #856:
URL: https://github.com/apache/flink-agents/pull/856#discussion_r3502125552
##########
docs/content/docs/faq/faq.md:
##########
@@ -87,7 +87,7 @@ This is important because:
- **For Java users on JDK 21+**: Async execution is available.
- **For Java users on JDK < 21**: Async execution is not available and falls back to synchronous execution.
-> **Cross-language async note**: Async execution for cross-language resources requires the pemja 0.5.7 fix, available in Flink 1.20.5+ / 2.0.2+ / 2.1.3+ / 2.2.1+. Current builds target Flink 2.2.0, so cross-language calls still run synchronously for now; this is resolved automatically once running on a Flink version that includes the fix.
+> **Cross-language async note**: Async execution for cross-language resources requires the pemja 0.5.7 fix, available in Flink 1.20.5+ / 2.0.2+ / 2.1.3+ / 2.2.1+ / 2.3+. Current builds target Flink 2.2.1, which includes the fix, so cross-language async is enabled by default; on older Flink versions it falls back to synchronous execution automatically.
Review Comment:
This line still reads `Current builds target Flink 2.2.1`, but the root `flink.version` now defaults to `2.3.0` (`pom.xml:44`) — looks like the version number didn't follow the default bump. Should it be `2.3.0`? The rest of the sentence (the fix is present, so async is on by default) holds either way.
--
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] [dist] Add Flink 2.3 distribution and make it the default [flink-agents]
Posted by "GreatEugenius (via GitHub)" <gi...@apache.org>.
GreatEugenius commented on PR #856:
URL: https://github.com/apache/flink-agents/pull/856#issuecomment-4861023797
Good catch, thanks @weiqingy — updated `faq.md` to `Current builds target Flink 2.3.0` to follow the default bump. Also rebased onto the latest `main`. Full build passes on JDK 11 across all 5 dist versions (1.20/2.0/2.1/2.2/2.3).
--
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] [dist] Add Flink 2.3 distribution and make it the default [flink-agents]
Posted by "GreatEugenius (via GitHub)" <gi...@apache.org>.
GreatEugenius commented on PR #856:
URL: https://github.com/apache/flink-agents/pull/856#issuecomment-4787541263
Hi, @wenjin272. Could you please review this PR? Thank you!
--
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] [dist] Add Flink 2.3 distribution and make it the default [flink-agents]
Posted by "wenjin272 (via GitHub)" <gi...@apache.org>.
wenjin272 merged PR #856:
URL: https://github.com/apache/flink-agents/pull/856
--
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