JNSimba commented on PR #678: URL: https://github.com/apache/doris-flink-connector/pull/678#issuecomment-5161886225
> Yes, sampling JDK 21 runtime coverage on a single Flink version should be sufficient. The package jobs for Flink 2.0, 2.1, and 2.2 are short and validate compilation and dependency compatibility, while the JDK 21 IT jobs are the main cost. I propose keeping the package matrix for all three versions, but running the JDK 21 IT and E2E coverage only on Flink 2.2. The E2E job is already limited to Flink 2.2, so this would remove the Flink 2.1 JDK 21 IT leg. If by "just one of them" you meant retaining only one test suite, either IT or E2E, please let me know which one you prefer and I will reduce it further. Yes, the build can retain all versions to ensure compilation succeeds. For ITcase and E2ECase, we only need to keep the flink2.2-jdk21 version—what do you think? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
