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.

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

Update hazelcast wm to 5.x.y compatibility (redux) [HZG-17] #135

Merged
merged 41 commits into from
May 28, 2024

Conversation

srknzl
Copy link
Member

@srknzl srknzl commented May 28, 2024

Redux of #127. It's reverted due to a merge related problem.

Prepare for release 5.0-SNAPSHOT

Uses tomcat 9.0.z because of https://hazelcast.atlassian.net/browse/SUP-420?focusedCommentId=97796

I changed the java version of PR builder to 17 because hazelcast 5.4 requires min 17

I tried spring session repl. guide locally with use-client:true/false and hz versions changing from 5.0 -> 5.5

Changes:

The main change is pom.xml. The new profiles are meant to be tested using https://jenkins.hazelcast.com/job/WM-pr-builder-different-versions-test/ (manual trigger).
There're refactors and copyright changes.
what I call refactor is basically about separation of jetty specific tests from others to be able to test jdk8 + tomcat 9 case (SUP-420 case, we don't know their jdk version but tomcat's min is 1.8)
some very small code changes to make dependencies work
and tiny bit of readme change

@srknzl srknzl changed the title Update hazelcast wm to 5.x.y compatibility [HZG-17] [redux] Update hazelcast wm to 5.x.y compatibility (redux) [HZG-17] May 28, 2024
@hasancelik hasancelik self-requested a review May 28, 2024 13:01
@srknzl srknzl merged commit 9e2fb06 into master May 28, 2024
2 checks passed
@srknzl srknzl deleted the update-hazelcast-wm-5.3.z branch May 28, 2024 13:11
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.

3 participants