Skip to content

Migrate OLFS code from JavaEE 8 to Jakarta EE 9 for Tomcat 11#223

Closed
ndp-opendap wants to merge 53 commits intomasterfrom
tomcat-11
Closed

Migrate OLFS code from JavaEE 8 to Jakarta EE 9 for Tomcat 11#223
ndp-opendap wants to merge 53 commits intomasterfrom
tomcat-11

Conversation

@ndp-opendap
Copy link
Copy Markdown
Contributor

@ndp-opendap ndp-opendap commented May 20, 2025

Here is a potential solution for HYRAX-1754

With this PR we move to Tomcat-11 and the java servlet-api-6.1

In migrating our code to use the java servlet-api-6.x(Where some group of deciders decided to change all the package names in the servlet-api. Nice, thanks for that.) we find there is a surprising amount of phaphphing about to do.

Update: This is working on RHEL-9 with Tomcat-11, and the Redis session manager.

We have to decide if it's "time to migrate", which is not insignificant because we have to bring libdap4, the new bes-deps, and the bes along for the ride to RHEL-9.

decided to change all the package names in the servlet-api.
Nice.
Thanks for that.
With this branch we move to catalina-11.0.7 and servlet-api-6.1
Copy link
Copy Markdown
Contributor

@dh-opendap dh-opendap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Need to determine if memcached session manager works with catalian-11.0.7 before merging.

@ndp-opendap
Copy link
Copy Markdown
Contributor Author

I think we should try it with a Tomcat 11 deployment! I have no idea if t works, just that it compiles.
But if we merge this then it will break the hyrax-docker production because that task is still using Tomcat 9 images

@dh-opendap
Copy link
Copy Markdown
Contributor

Agreed, let's try this with memcached.

Copy link
Copy Markdown
Contributor

@dh-opendap dh-opendap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woth trying with memcached session manager

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants