Sxnan opened a new pull request, #413: URL: https://github.com/apache/flink-agents/pull/413
<!-- * 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: #389 ### Purpose of change This PR introduces support for multiple JDK versions in Flink Agents, allowing users to choose between JDK 11 and JDK 17 based on their feature requirements. **Key changes:** - Changed the default target Java version from 17 to 11 in the root `pom.xml`, making JDK 11 the baseline for most modules - Set the MCP module (`integrations/mcp`) to explicitly require JDK 17, as the MCP SDK depends on Java 17 features - Added a new `java-11-target` Maven profile in `api/pom.xml` to generate a `jdk11` classifier JAR that excludes `MCPServer.java` (which depends on Java 17) - Updated the release script (`deploy_staging_jars.sh`) to deploy both the default JDK 17 version and the `jdk11` classifier version ### Tests - Existing unit tests cover the functionality - Manual verification of the Maven build with both JDK 11 and JDK 17 ### API No public API changes. This PR only affects build configuration and JDK compatibility. ### Documentation <!-- Do not remove this section. Check the proper box only. --> - [x] `doc-needed` <!-- Your PR changes impact docs --> - [ ] `doc-not-needed` <!-- Your PR changes do not impact docs --> -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
