I was going to say that, as far as I know, all modules should be set to 17 now. Anyway, the task is to verify that assumption is true.
On Wed, Jul 30, 2025 at 4:43 PM Yeser Amer <[email protected]> wrote: > Tibor, > > Thank you for your initiative, can you please clarify: > > > Phase 1: > > - Setting the release property: > > <maven.compiler.release>17</maven.compiler.release>. > > That means that we currently have some modules with JDK version < 17, > right? > > > Phase 2: > > - Update the KIE CI to use Java 21 by default. This means: > > - Making sure PR checks run with Java 21. > > - Making sure the release builds run with Java 21. > > If I understood correctly, we have to support both JDK 17 and JDK 21 for a > while. Should we expect to have 2 subsets of PR Checks, one that runs > against JDK 17 and one that compiles against JDK 21 (optaplanner is > correctly doing that against JDK 17 and 20)? > > Thanks > > Yeser > > > On 2025/07/30 11:54:23 Tibor Zimányi wrote: > > Hi everyone, > > > > as Java 21 is the latest Java LTS available, already for some time, I > would > > like to propose that the code should be buildable and runnable with Java > > 21, while still retaining compatibility with Java 17. > > > > The work should be split into two phases like this: > > Phase 1: > > - Setting the release property: > > <maven.compiler.release>17</maven.compiler.release>. > > - This should make sure the code works with Java 17. > > - This should make sure that the existing CI still works the same way > > and is able to build releases etc., because even if we have Java 17 as a > > build environment, with the setting, it should work. > > - Fixing the problems that occur while trying to build the repositories > > with Java 21 and are caused by Java 21. > > - Fixing the problems that occur while running tests with Java 21 and are > > caused by Java 21. > > - Updating the images we publish and need JDK to contain JDK 21 instead > of > > 17. > > - Target of the work are all Apache KIE repositories, mainly: > > incubator-kie-drools, incubator-kie-optaplanner, > > incubator-kie-kogito-runtimes, incubator-kie-kogito-apps, > > incubator-kie-kogito-examples, incubator-kie-tools. > > > > Phase 2: > > - Update the KIE CI to use Java 21 by default. This means: > > - Making sure PR checks run with Java 21. > > - Making sure the release builds run with Java 21. > > > > What do you think, please? I plan to open a vote on Monday based on the > > feedback of this discussion. Finding people to do this work is already > > done, at least for Phase 1. > > > > Best regards, > > Tibor > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
