search

Netflix / mantis

A platform that makes it easy for developers to build realtime, cost-effective, operations-focused applications
Apache License 2.0
1.42k stars 202 forks source link

Upgrade JDK version to 17 #355

Open sundargates opened 1 year ago

sundargates commented 1 year ago

Context

We want mantis to be on Java17. Unfortunately, this would require several gradle changes. So, the goal of this task is to make these changes and ensure the builds succeed.

Adithya2907 commented 1 year ago

I think one of the reasons it fails for higher versions of JDK is the DatatypeConverter package. As this answer on StackOverflow explains:

The module javax.xml.bind has been put outside the jdk as of Java 9.

Also, as part of the donation of Java EE to the Eclipse Foundation, its new name is Jakarta EE and accordingly the namespace was renamed to Jakarta. So you have to make the following modifications :
sundargates commented 1 year ago

@Adithya2907 would you like to pick this up?

Adithya2907 commented 1 year ago

Yeah, sure I'd like to pick this up.

cypherean commented 1 year ago

Hey @Adithya2907 Are you working on this?

Adithya2907 commented 1 year ago

Hi @cypherean , yes, I'm working on this. I've been trying to understand the codebase till now, and now I'll be migrating it to JDK 17. I'm seeing which all packages and libraries are failing due to the migration first