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

Use Hazelcast 5.5.0 #170

Closed
wants to merge 37 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
02862d8
Use newest Hazelcast
TomaszGaweda Dec 4, 2024
9f22d01
OSS doesn't have the patch version
TomaszGaweda Dec 4, 2024
e8fc48b
Update Tomcat dep
TomaszGaweda Dec 18, 2024
b687419
Many dep updates
TomaszGaweda Dec 18, 2024
832f58b
Cleanups
TomaszGaweda Dec 18, 2024
f953439
Cleanups
TomaszGaweda Dec 18, 2024
166f2cc
Waiting to cluster moved to only some tests. Added preferIPv4 in othe…
TomaszGaweda Dec 18, 2024
d957458
Some Cleanup
TomaszGaweda Jan 10, 2025
acf3689
Some Cleanup
TomaszGaweda Jan 14, 2025
60d843f
Hazelcast is inherently AP - so use RetryingTest, trying to mitigate …
TomaszGaweda Jan 14, 2025
e480457
Checkstyle
TomaszGaweda Jan 14, 2025
0b64915
Revert "Hazelcast is inherently AP - so use RetryingTest, trying to m…
TomaszGaweda Jan 14, 2025
e7db9c8
Retry in other way
TomaszGaweda Jan 14, 2025
b208a6c
Activate master by default
TomaszGaweda Jan 14, 2025
a190a55
Add preferIpV4Stack by default
TomaszGaweda Jan 14, 2025
30b7e4a
Some reverts
TomaszGaweda Jan 14, 2025
4db451b
Use again HTS
TomaszGaweda Jan 14, 2025
52bed57
Cleanup
TomaszGaweda Jan 14, 2025
38d2935
Temp
TomaszGaweda Jan 15, 2025
4421f5b
Merge branch 'master' into TomaszGaweda-patch-1
TomaszGaweda Jan 15, 2025
543b823
Removing not necessary change
TomaszGaweda Jan 15, 2025
351bab6
Apply suggestions from code review
TomaszGaweda Jan 15, 2025
d98efa0
Remove not used code
TomaszGaweda Jan 15, 2025
b8909dc
Merge remote-tracking branch 'origin/TomaszGaweda-patch-1' into Tomas…
TomaszGaweda Jan 15, 2025
e377d8d
Remove test code
TomaszGaweda Jan 15, 2025
8a9695c
Minor dependencies bump
TomaszGaweda Jan 15, 2025
5bc6115
Cleanup
TomaszGaweda Jan 15, 2025
9d42030
OK don't use snapshot yet
TomaszGaweda Jan 15, 2025
883e236
Few tweaks
TomaszGaweda Jan 15, 2025
a63466d
Few tweaks
TomaszGaweda Feb 13, 2025
cbb9e5a
Merge branch 'master' into TomaszGaweda-patch-1
TomaszGaweda Feb 13, 2025
75cc123
Merge branch 'master' into TomaszGaweda-patch-1
TomaszGaweda Feb 13, 2025
e58a73a
Merge remote-tracking branch 'origin/TomaszGaweda-patch-1' into Tomas…
TomaszGaweda Feb 13, 2025
7adffa0
Change default version
TomaszGaweda Feb 13, 2025
b4ac7b1
SHow results
TomaszGaweda Feb 14, 2025
6314647
Output in console
TomaszGaweda Feb 14, 2025
77ff7c1
More config from core, Restore SUP-420 case
TomaszGaweda Feb 14, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion .github/workflows/pr-builder.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:

jobs:
build:
runs-on: ubuntu-latest
runs-on: ubicloud-standard-8

steps:
- name: Get all the letters, words and paragraphs needed for the spell!
Expand All @@ -23,3 +23,10 @@ jobs:
- name: Verify
run: |
mvn verify -B -Dmaven.test.failure.ignore=false

- name: Publish Test results
uses: EnricoMi/publish-unit-test-result-action@567cc7f8dcea3eba5da355f6ebc95663310d8a07
if: ${{ !cancelled() }}
with:
files: "**/TEST-*.xml"
large_files: true
11 changes: 11 additions & 0 deletions .officialRun/Template JUnit.run.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
<component name="ProjectRunConfigurationManager">
<configuration default="true" type="JUnit" factoryName="JUnit">
<option name="MAIN_CLASS_NAME" value="" />
<option name="METHOD_NAME" value="" />
<option name="TEST_OBJECT" value="class" />
<option name="VM_PARAMETERS" value="-ea -Djava.net.preferIPv4Stack=true" />
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added because we want to use java.net.preferIPv4Stack always

<method v="2">
<option name="Make" enabled="true" />
</method>
</configuration>
</component>
48 changes: 0 additions & 48 deletions build.xml

This file was deleted.

148 changes: 74 additions & 74 deletions pom.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<!--
~ Copyright 2024 Hazelcast Inc.
~ Copyright 2025 Hazelcast Inc.
~
~ Licensed under the Hazelcast Community License (the "License"); you may not use
~ this file except in compliance with the License. You may obtain a copy of the
Expand Down Expand Up @@ -31,35 +31,38 @@
<maven.build.timestamp.format>yyyyMMdd</maven.build.timestamp.format>
<timestamp>${maven.build.timestamp}</timestamp>

<hazelcast.version>5.4.0</hazelcast.version>
<hazelcast.version>5.5.0</hazelcast.version>

<log4j.version>3.0.0-beta2</log4j.version>
<junit.version>4.13.2</junit.version>
<junit.version>5.11.4</junit.version>
<assertj.version>3.26.0</assertj.version>
<opentest4j.version>1.3.0</opentest4j.version>

<jsp.api.version>4.0.0</jsp.api.version>
<servlet.api.version>6.1.0</servlet.api.version>
<org.springframework.version>6.1.14</org.springframework.version>
<spring-security.version>6.3.4</spring-security.version>
<jetty.maven.plugin.version>11.0.21</jetty.maven.plugin.version>

<org.springframework.version>6.2.1</org.springframework.version>
<spring-security.version>6.4.2</spring-security.version>
<jetty.maven.plugin.version>11.0.24</jetty.maven.plugin.version>
<tomcat.version>11.0.2</tomcat.version>
<httpclient.version>4.5.13</httpclient.version>

<maven.compiler.plugin.version>3.13.0</maven.compiler.plugin.version>
<maven.jar.plugin.version>2.4</maven.jar.plugin.version>
<maven.jar.plugin.version>3.4.2</maven.jar.plugin.version>
<maven.source.plugin.version>3.3.1</maven.source.plugin.version>
<maven.bundle.plugin.version>5.1.9</maven.bundle.plugin.version>
<maven.bundle.plugin.version>6.0.0</maven.bundle.plugin.version>
<maven.animal.sniffer.plugin.version>1.14</maven.animal.sniffer.plugin.version>
<maven.gpg.plugin.version>3.2.4</maven.gpg.plugin.version>
<maven.javadoc.plugin.version>2.9</maven.javadoc.plugin.version>
<maven.javadoc.plugin.version>3.11.2</maven.javadoc.plugin.version>
<bytebuddy.version>1.14.17</bytebuddy.version>

<maven.surefire.plugin.version>2.18.1</maven.surefire.plugin.version>
<maven.surefire.plugin.version>3.5.2</maven.surefire.plugin.version>
<maven.checkstyle.plugin.version>3.4.0</maven.checkstyle.plugin.version>
<maven.spotbugs.plugin.version>4.8.5.0</maven.spotbugs.plugin.version>
<maven.sonar.plugin.version>3.3.0.603</maven.sonar.plugin.version>
<maven.spotbugs.plugin.version>4.8.6.6</maven.spotbugs.plugin.version>
<maven.sonar.plugin.version>5.0.0.4389</maven.sonar.plugin.version>
<maven.jacoco.plugin.version>0.7.9</maven.jacoco.plugin.version>
<maven-release-plugin-version>3.0.1</maven-release-plugin-version>
<mockito.version>5.12.0</mockito.version>
</properties>

<licenses>
Expand Down Expand Up @@ -139,23 +142,17 @@
<artifactId>maven-surefire-plugin</artifactId>
<version>${maven.surefire.plugin.version}</version>
<configuration combine.self="override">
<redirectTestOutputToFile>true</redirectTestOutputToFile>
<redirectTestOutputToFile>false</redirectTestOutputToFile>
<useFile>false</useFile>
<runOrder>failedfirst</runOrder>

<!-- 1C means 1 process per cpu core -->
<!-- forkCount>1C</forkCount>
<reuseForks>true</reuseForks>

<threadCount>1</threadCount>
<perCoreThreadCount>true</perCoreThreadCount>
<parallel>methods</parallel -->

<!-- the argLine variable is needed for JaCoco -->
<trimStackTrace>false</trimStackTrace>
<argLine>
-Xms128m -Xmx2G
-Dhazelcast.phone.home.enabled=false
-Dhazelcast.mancenter.enabled=false
-Dhazelcast.test.use.network=false
-Djava.net.preferIPv4Stack=true
-Dhazelcast.logging.type=log4j2
</argLine>
</configuration>
</plugin>
Expand All @@ -176,7 +173,6 @@
<configLocation>${main.basedir}/checkstyle/checkstyle.xml</configLocation>
<suppressionsLocation>${main.basedir}/checkstyle/suppressions.xml</suppressionsLocation>
<headerLocation>${main.basedir}/checkstyle/ClassHeader.txt</headerLocation>
<enableRSS>false</enableRSS>
<linkXRef>true</linkXRef>
<consoleOutput>true</consoleOutput>
<failsOnError>true</failsOnError>
Expand Down Expand Up @@ -265,10 +261,10 @@
<configuration>
<webAppSourceDirectory>${basedir}/src/test/webapp</webAppSourceDirectory>
<useTestScope>true</useTestScope>
<webAppConfig>
<webApp>
<contextPath>/</contextPath>
<descriptor>${basedir}/src/test/webapp/WEB-INF/web.xml</descriptor>
</webAppConfig>
</webApp>
<stopPort>9966</stopPort>
<stopKey>stop</stopKey>
</configuration>
Expand Down Expand Up @@ -303,6 +299,7 @@
<version>${servlet.api.version}</version>
<scope>provided</scope>
</dependency>

<!-- Needed to make Tomcat log in tests-->
<dependency>
<groupId>org.apache.logging.log4j</groupId>
Expand Down Expand Up @@ -382,6 +379,19 @@
<artifactId>httpclient</artifactId>
<version>${httpclient.version}</version>
<scope>test</scope>
<exclusions>
Copy link
Contributor

Choose a reason for hiding this comment

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

Why exclusions? Maybe left a comment?

<!-- We want to remove CL from classpath and use SLF4j -->
<exclusion>
<groupId>commons-logging</groupId>
<artifactId>commons-logging</artifactId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>jcl-over-slf4j</artifactId>
<version>2.0.9</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.apache.tomcat</groupId>
Expand All @@ -408,11 +418,16 @@
<scope>test</scope>
</dependency>
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
<groupId>org.junit.jupiter</groupId>
<artifactId>junit-jupiter-engine</artifactId>
<version>${junit.version}</version>
<scope>test</scope>
<optional>true</optional>
<scope>compile</scope>
</dependency>
<dependency>
<groupId>org.junit.vintage</groupId>
<artifactId>junit-vintage-engine</artifactId>
<version>${junit.version}</version>
<scope>compile</scope>
</dependency>
<dependency>
<groupId>org.apache.logging.log4j</groupId>
Expand Down Expand Up @@ -442,6 +457,24 @@
<version>${assertj.version}</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.opentest4j</groupId>
<artifactId>opentest4j</artifactId>
<version>${opentest4j.version}</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.awaitility</groupId>
<artifactId>awaitility</artifactId>
<version>4.2.2</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.mockito</groupId>
<artifactId>mockito-core</artifactId>
<version>${mockito.version}</version>
<scope>test</scope>
</dependency>
</dependencies>
<profiles>
<profile>
Expand Down Expand Up @@ -475,6 +508,7 @@
-Dhazelcast.version.check.enabled=false
-Dhazelcast.mancenter.enabled=false
-Dhazelcast.test.use.network=false
-Djava.net.preferIPv4Stack=true
</argLine>
</properties>
<build>
Expand All @@ -496,7 +530,7 @@
</plugin>

<plugin>
<groupId>org.codehaus.mojo</groupId>
<groupId>org.sonarsource.scanner.maven</groupId>
<artifactId>sonar-maven-plugin</artifactId>
<version>${maven.sonar.plugin.version}</version>
</plugin>
Expand Down Expand Up @@ -612,6 +646,7 @@
<hazelcast.version>5.3.7</hazelcast.version>
</properties>
</profile>

<profile>
<id>5.4.z</id>
<properties>
Expand All @@ -620,47 +655,18 @@
</profile>
<profile>
<id>master</id>
<dependencies>
<dependency>
<groupId>org.opentest4j</groupId>
<artifactId>opentest4j</artifactId>
<version>${opentest4j.version}</version>
</dependency>
<dependency>
<groupId>org.mockito</groupId>
<artifactId>mockito-core</artifactId>
<version>${mockito.version}</version>
<scope>test</scope>
</dependency>
</dependencies>
<activation>
<activeByDefault>true</activeByDefault>
</activation>
<properties>
<hazelcast.version>5.5.0-SNAPSHOT</hazelcast.version>
<mockito.version>5.12.0</mockito.version>
<opentest4j.version>1.3.0</opentest4j.version>
<hazelcast.version>5.5.0</hazelcast.version>
</properties>
<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<version>${maven.surefire.plugin.version}</version>
<configuration combine.self="override">
<parallel>none</parallel>
<redirectTestOutputToFile>true</redirectTestOutputToFile>
<argLine>-Xms128m -Xmx2G
-Dhazelcast.phone.home.enabled=false
-Dhazelcast.mancenter.enabled=false
-Dhazelcast.test.use.network=true
</argLine>
</configuration>
</plugin>
</plugins>
</build>
</profile>

<profile>
<!-- SUP-420 case. Tomcat 9.0.z and 5.3.z hazelcast. Using jdk 1.8 because tomcat supports 1.8.
Therefore, we have to exclude jetty tests from compiling/running and jetty classes from classpath
because they require jdk 11. -->
<!-- SUP-420 case. Tomcat 9.0.z and 5.3.z hazelcast. Using jdk 1.8 because tomcat supports 1.8.
Therefore, we have to exclude jetty tests from compiling/running and jetty classes from classpath
because they require jdk 11. -->
<id>tomcat-jdk8</id>
<properties>
<hazelcast.version>5.3.7</hazelcast.version>
Expand Down Expand Up @@ -700,12 +706,6 @@
<configuration combine.self="override">
<redirectTestOutputToFile>true</redirectTestOutputToFile>
<runOrder>failedfirst</runOrder>
<argLine>
-Xms128m -Xmx2G
-Dhazelcast.phone.home.enabled=false
-Dhazelcast.mancenter.enabled=false
-Dhazelcast.test.use.network=false
</argLine>
<excludes>
<exclude>
com.hazelcast.wm.test.jetty.*
Expand Down
Loading