GSIP 232 - Upgrade GeoServer and GeoWebCache Build to Java 17

Hi all,

As mentioned in Jody’s “GS3 Milestone 1 Updates” topic, we’re ramping up on the GeoServer crowdfunding work.

A pre-requisite for it is establishing Java 17 as the baseline version for the build, hence I’m making this proposal to cover both GeoServer and GeoWebCache:

It goes in tandem with the GeoTools change proposal here: Migrate GeoTools to Java 17 · geotools/geotools Wiki · GitHub

The preliminary work is done and validated. What remains is to make the Error Prone plugin happy as is reports a lot more issues.

Please review and share your feedback.

Thanks,

Gabe

+1

+0
I support the motion but unfortunately with my Java knowledge I cannot judge if it is ready to merge.

-Jukka-

+1

Ian

On Tue, 3 Jun 2025, 18:08 Jukka Rahkonen via OSGeo Discourse, <noreply@discourse.osgeo.org> wrote:

jratike80
June 3

+0
I support the motion but unfortunately with my Java knowledge I cannot judge if it is ready to merge.

-Jukka-


Visit Topic or reply to this email to respond.

To unsubscribe from these emails, click here.

+1

Cheers,
Torben

On Thu, May 29, 2025 at 5:56 AM Gabriel Roldan via OSGeo Discourse <noreply@discourse.osgeo.org> wrote:

groldan
May 29

Hi all,

As mentioned in Jody’s “GS3 Milestone 1 Updates” topic, we’re ramping up on the GeoServer crowdfunding work.

A pre-requisite for it is establishing Java 17 as the baseline version for the build, hence I’m making this proposal to cover both GeoServer and GeoWebCache:

GitHub

GSIP 232

Official GeoServer repository. Contribute to geoserver/geoserver development by creating an account on GitHub.

It goes in tandem with the GeoTools change proposal here: Migrate GeoTools to Java 17 · geotools/geotools Wiki · GitHub

The preliminary work is done and validated. What remains is to make the Error Prone plugin happy as is reports a lot more issues.

Please review and share your feedback.

Thanks,

Gabe


Visit Topic or reply to this email to respond.

You are receiving this because you enabled mailing list mode.

To unsubscribe from these emails, click here.

+0

Gabe I am marking this motion as passed, and have updated the text to reflect GeoTools 34.x development, and GeoTools 35.x development targets.

Hi all,

sorry for cross-posting but this is a coordinated effort across GeoTools/GeoServer/GeoWebcache.

The pull requests to upgrade the builds to Java 17 and pass the new errorprone/PMD checks are ready:

The GeoTools and GeoWebCache ones can be applied independently. The GeoServer one depends on the
GeoTools pr to be merged first, mostly due to the need to fix the usage of varargs in the GridCoverageReader/Writer
class hierarchy.

Thanks everyone for voting and reviewing. Let me know if you have any questions or further concerns.

Cheers,
Gabe

1 Like

Thanks Gabe, I have been reviewing, making small changes to the docs and setting standalone gwc up for local testing (against a local geoserver).

Please see individual PRs for feedback (and no force push so my work is not lost :smiley: )

It looks like all three PRs have received a review and feedback addressed.

Hope to merge this shortly!

Does anyone else wish to look before we start?

Okay:

  • Noticed that the code formatting has not been applied on all modules, so when those were changed as part of Java 17 they would get formatted.

    Action: Apply formatting on modules that were missed and rebase, possibly introducing a -Dall activation to GeoServer to make this kind of thing easier.

  • ErrorProne identified width and height parameters being swapped when calling between methods. Appears just to be a mistake.

    Action: Jira

On Thu, Jun 26, 2025 at 5:17 PM Jody Garnett via OSGeo Discourse <noreply@discourse.osgeo.org> wrote:

Does anyone else wish to look before we start?

If you can wait Monday I’ll be happy to have a look

Cheers
Andrea