Skip to content
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

Jump to bottom

[Java] JDK 22 Support #40680

Closed
danepitkin opened this issue Mar 19, 2024 · 8 comments
Closed

[Java] JDK 22 Support #40680

danepitkin opened this issue Mar 19, 2024 · 8 comments
Assignees
@danepitkin
Labels
Component: Java Type: enhancement
Milestone
16.0.0

Comments

@danepitkin
Copy link
Member

Describe the enhancement requested

Verify JDK 22 and enable it in CI jobs.

Component(s)

Java
@danepitkin
Copy link
Member Author

We need to wait for the Eclipse Temurin 22 release and then for the maven/temurin docker release before we can add to CI.

@vibhatha
Copy link
Collaborator

I can take a look at this.

@danepitkin
Copy link
Member Author

Here is the link to the maven docker registry: https://hub.docker.com/_/maven

It will probably take a week or two before the container is available.

@danepitkin danepitkin added this to the 16.0.0 milestone Mar 19, 2024
@vibhatha
Copy link
Collaborator

take

@danepitkin
Copy link
Member Author

For tracking purposes adoptium/temurin#35

@github-actions github-actions bot mentioned this issue Apr 5, 2024
Merged
@danepitkin danepitkin self-assigned this Apr 5, 2024
@danepitkin
Copy link
Member Author

Hey @vibhatha , I went ahead and created the PR!

lidavidm pushed a commit that referenced this issue Apr 5, 2024
@danepitkin
GH-40680: [Java] Test JDK 22 in CI (#41038)
be3baf2 
### Rationale for this change

JDK 22 is released. Let's ensure Arrow java CI tests this version successfully.

### What changes are included in this PR?

* Add JDK 22 to CI

### Are these changes tested?

Tested by CI jobs

### Are there any user-facing changes?

No
* GitHub Issue: #40680

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
@lidavidm
Copy link
Member

lidavidm commented Apr 6, 2024

Issue resolved by pull request 41038
#41038

@lidavidm lidavidm closed this as completed Apr 6, 2024
@vibhatha
Copy link
Collaborator

vibhatha commented Apr 7, 2024

Hey @vibhatha , I went ahead and created the PR!

Thanks Dane, it could have taken me a little longer. Apologies I couldn't review this PR before it got merged. I reviewed it later, looks good to me.

tolleybot pushed a commit to tmct/arrow that referenced this issue May 2, 2024
@danepitkin @tolleybot
apacheGH-40680: [Java] Test JDK 22 in CI (apache#41038)
69bccaf 
### Rationale for this change

JDK 22 is released. Let's ensure Arrow java CI tests this version successfully.

### What changes are included in this PR?

* Add JDK 22 to CI

### Are these changes tested?

Tested by CI jobs

### Are there any user-facing changes?

No
* GitHub Issue: apache#40680

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
tolleybot pushed a commit to tmct/arrow that referenced this issue May 4, 2024
@danepitkin @tolleybot
apacheGH-40680: [Java] Test JDK 22 in CI (apache#41038)
253a8cb 
### Rationale for this change

JDK 22 is released. Let's ensure Arrow java CI tests this version successfully.

### What changes are included in this PR?

* Add JDK 22 to CI

### Are these changes tested?

Tested by CI jobs

### Are there any user-facing changes?

No
* GitHub Issue: apache#40680

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
rok pushed a commit to tmct/arrow that referenced this issue May 8, 2024
@danepitkin @rok
apacheGH-40680: [Java] Test JDK 22 in CI (apache#41038)
5544f3b 
### Rationale for this change

JDK 22 is released. Let's ensure Arrow java CI tests this version successfully.

### What changes are included in this PR?

* Add JDK 22 to CI

### Are these changes tested?

Tested by CI jobs

### Are there any user-facing changes?

No
* GitHub Issue: apache#40680

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
rok pushed a commit to tmct/arrow that referenced this issue May 8, 2024
@danepitkin @rok
apacheGH-40680: [Java] Test JDK 22 in CI (apache#41038)
7fadc55 
### Rationale for this change

JDK 22 is released. Let's ensure Arrow java CI tests this version successfully.

### What changes are included in this PR?

* Add JDK 22 to CI

### Are these changes tested?

Tested by CI jobs

### Are there any user-facing changes?

No
* GitHub Issue: apache#40680

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
vibhatha pushed a commit to vibhatha/arrow that referenced this issue May 25, 2024
@danepitkin @vibhatha
apacheGH-40680: [Java] Test JDK 22 in CI (apache#41038)
56288ce 
### Rationale for this change

JDK 22 is released. Let's ensure Arrow java CI tests this version successfully.

### What changes are included in this PR?

* Add JDK 22 to CI

### Are these changes tested?

Tested by CI jobs

### Are there any user-facing changes?

No
* GitHub Issue: apache#40680

Authored-by: Dane Pitkin <dane@voltrondata.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danepitkin danepitkin

Labels
Component: Java Type: enhancement
Projects
None yet
Milestone
16.0.0
Development

No branches or pull requests
3 participants
@lidavidm @vibhatha @danepitkin