Skip to content

AVRO-4191 - include tests on JDK 25 #3685

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
apupier wants to merge 10 commits into
base: main
Choose a base branch
from
Open

AVRO-4191 - include tests on JDK 25 #3685

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
31f2de5
AVRO-4191 - include tests on JDK 25
apupier Mar 9, 2026
e902095
Trigger the build
martin-g Mar 9, 2026
e375497
Revert "Trigger the build"
martin-g Mar 9, 2026
9fada45
Yaml formatting to fix the CI workflows
martin-g Mar 9, 2026
266373a
Use a matrix to build with all supported JDK versions
martin-g Mar 9, 2026
3bc2f53
Avro 1.13.x will require JDK 17+
martin-g Mar 9, 2026
fb56cf4
Use Checkstyle 11.x to be able to build with JDK 17
martin-g Mar 9, 2026
cacd414
AVRO-4191 - Update Java Maven toolchain in Build.MD
apupier Mar 9, 2026
b62da48
AVRo-4191 - Fix naming of github steps related to java
apupier Mar 9, 2026
7c0f033
AVRo-4191 - remove JDK 11 from test execution
apupier Mar 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions .github/workflows/codeql-java-analysis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,15 +70,14 @@ jobs:
# queries: ./path/to/local/query, your-org/your-repo/queries@main
queries: +security-and-quality

- name: 'Setup Temurin JDK 8, 11, 17 & 21'
- name: 'Setup Temurin JDK 8, 11, 17, 21 & 25'
Comment thread
apupier marked this conversation as resolved.
Outdated
uses: actions/setup-java@v5
with:
distribution: 'temurin'
java-version: |
8
11
17
21
25

- name: 'Setup Maven 3.9.11'
uses: stCarolas/setup-maven@d6af6abeda15e98926a57b5aa970a96bb37f97d1 # v5
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/java-publish-snapshot.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
restore-keys: |
${{ runner.os }}-maven-

- name: 'Setup Temurin JDK 8, 11, 17 & 21'
- name: 'Setup Temurin JDK 8, 11, 17, 21 & 25'
uses: actions/setup-java@v5
with:
distribution: 'temurin'
Expand All @@ -52,6 +52,7 @@ jobs:
11
17
21
25

- name: 'Setup Maven'
uses: stCarolas/setup-maven@d6af6abeda15e98926a57b5aa970a96bb37f97d1 # v5
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/maven4.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,15 +50,14 @@ jobs:
restore-keys: |
${{ runner.os }}-maven-build-cache

- name: 'Setup Temurin JDK 8, 11, 17 & 21'
- name: 'Setup Temurin JDK 8, 11, 17, 21 & 25'
uses: actions/setup-java@v5
with:
distribution: 'temurin'
java-version: |
8
11
17
21
25

- name: Setup Maven 4
uses: stCarolas/setup-maven@d6af6abeda15e98926a57b5aa970a96bb37f97d1 # v5
Expand Down
26 changes: 14 additions & 12 deletions .github/workflows/test-lang-java.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,10 @@ jobs:
os:
- ubuntu-latest
- ubuntu-24.04-arm
jdk:
- 17
- 21
- 25
steps:
- name: "Checkout sourcecode"
uses: actions/checkout@v6
Expand All @@ -54,14 +58,11 @@ jobs:
restore-keys: |
${{ runner.os }}-maven-

- name: "Setup Temurin JDK 11, 17 & 21"
- name: "Setup Temurin JDK ${{ matrix.jdk }}"
uses: actions/setup-java@v5
with:
distribution: "temurin"
java-version: |
11
17
21
java-version: ${{ matrix.jdk }}

- name: "Setup Maven 3.9.11"
uses: stCarolas/setup-maven@d6af6abeda15e98926a57b5aa970a96bb37f97d1 # v5
Expand Down Expand Up @@ -94,6 +95,10 @@ jobs:
os:
- ubuntu-latest
- ubuntu-24.04-arm
jdk:
- 17
- 21
- 25

steps:
- uses: actions/checkout@v6
Expand All @@ -106,14 +111,11 @@ jobs:
restore-keys: |
${{ runner.os }}-maven-

- name: "Setup Temurin JDK 11, 17 & 21"
- name: "Setup Temurin JDK ${{ matrix.jdk }}"
uses: actions/setup-java@v5
with:
distribution: "temurin"
java-version: |
11
17
21
java-version: ${{ matrix.jdk }}

- name: "Setup Maven 3.9.11"
uses: stCarolas/setup-maven@d6af6abeda15e98926a57b5aa970a96bb37f97d1 # v5
Expand Down Expand Up @@ -164,10 +166,10 @@ jobs:
working-directory: .
run: mvn -B install -PskipQuality

- name: "Generate Interop Data using Java 11, 17 & 21"
- name: "Generate Interop Data using Java 11, 17, 21 & 25"
working-directory: lang/java/interop-data-test
run: mvn -B verify -Pgenerate-test-data

- name: "Run Interop Tests using Java 11, 17 & 21"
- name: "Run Interop Tests using Java 11, 17, 21 & 25"
working-directory: lang/java/interop-data-test
run: mvn -B verify -Pcheck-test-data
16 changes: 16 additions & 0 deletions lang/java/avro/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,22 @@
<cloneProjectsTo>${project.build.directory}/it-jdk-21</cloneProjectsTo>
</configuration>
</execution>

<execution>
<id>Run all tests under Java 25</id>
<goals>
<goal>run</goal>
</goals>
<configuration>
<filterProperties>
<integrationTestingJDK>[25,26)</integrationTestingJDK>
</filterProperties>
<properties>
<maven.compiler.release>25</maven.compiler.release>
</properties>
<cloneProjectsTo>${project.build.directory}/it-jdk-25</cloneProjectsTo>
</configuration>
</execution>
</executions>
</plugin>

Expand Down
2 changes: 1 addition & 1 deletion pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@
</excludes>
</enforceBytecodeVersion>
<requireJavaVersion>
<version>21</version>
<version>17</version>
</requireJavaVersion>
<requireMavenVersion>
<version>[3.9.6,)</version>
Expand Down
Loading