Skip to content

Commit

Permalink
HBASE-27858 Update surefire version to 3.1.0 and use SurefireForkNode…
Browse files Browse the repository at this point in the history
…Factory (apache#5234)

Surefire version updated from 3.0.0-M6 -> 3.1.0.

SurefireForkNodeFactory is a new strategy to control how the forked
nodes communicate with the main maven process. It uses a tcp channel
instead of pipes and fixes some corrupted messages seen in the s390x
build.

Signed-off-by: Duo Zhang <zhangduo@apache.org>
  • Loading branch information
jonathan-albrecht-ibm committed May 13, 2023
1 parent bee09bb commit 67612cc
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -880,7 +880,7 @@
<findbugs-annotations.version>1.3.9-1</findbugs-annotations.version>
<spotbugs.version>4.7.3</spotbugs.version>
<spotbugs.maven.version>4.7.2.1</spotbugs.maven.version>
<surefire.version>3.0.0-M6</surefire.version>
<surefire.version>3.1.0</surefire.version>
<wagon.ssh.version>2.12</wagon.ssh.version>
<xml.maven.version>1.0.1</xml.maven.version>
<spotless.version>2.27.2</spotless.version>
Expand Down Expand Up @@ -1836,6 +1836,7 @@
<trimStackTrace>false</trimStackTrace>
<skip>${surefire.skipFirstPart}</skip>
<forkCount>${surefire.firstPartForkCount}</forkCount>
<forkNode implementation="org.apache.maven.plugin.surefire.extensions.SurefireForkNodeFactory"/>
<!--
The counter in HBaseTestAppender will be broken if we set reuseForks to true, be
careful when you want to change this value. See HBASE-26947 for more details.
Expand Down Expand Up @@ -1890,6 +1891,7 @@
-->
<reuseForks>false</reuseForks>
<forkCount>${surefire.secondPartForkCount}</forkCount>
<forkNode implementation="org.apache.maven.plugin.surefire.extensions.SurefireForkNodeFactory"/>
<groups>${surefire.secondPartGroups}</groups>
<forkedProcessTimeoutInSeconds>${surefire.timeout}</forkedProcessTimeoutInSeconds>
</configuration>
Expand Down

0 comments on commit 67612cc

Please sign in to comment.