[tinkerpop] branch master updated: Reverted to older version of surefire for performance reasons

[sp...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


The following commit(s) were added to refs/heads/master by this push:
     new 9acaaed  Reverted to older version of surefire for performance reasons
9acaaed is described below

commit 9acaaed1adbb2fa0fd831758f2c2d64d8ebe436c
Author: Stephen Mallette <st...@amazon.com>
AuthorDate: Wed Apr 28 08:10:57 2021 -0400

    Reverted to older version of surefire for performance reasons
    
    Using same version as 3.4-dev again. CTR
---
 pom.xml | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/pom.xml b/pom.xml
index 24b2ca4..5d3977b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -480,17 +480,19 @@ limitations under the License.
                     <artifactId>maven-deploy-plugin</artifactId>
                     <version>2.8.2</version>
                 </plugin>
+                <!-- reverted surefire to 2.21.0 for performance reasons. 2.22.2 required forkCount=1 and
+                     reuseForks=true which seemed to more the double the build time. there are only milestone
+                     releases after 2.22.2 so rather than rely on those at this time, i guess we just stick with
+                     2.21.0 -->
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-surefire-plugin</artifactId>
-                    <version>2.22.2</version>
+                    <version>2.21.0</version>
                     <configuration>
                         <argLine>
                             -Dlog4j.configuration=${log4j-test.properties} -Dbuild.dir=${project.build.directory} -Dis.testing=true -Djava.net.preferIPv4Stack=true ${suresafeArgs}
                         </argLine>
                         <trimStackTrace>false</trimStackTrace>
-                        <forkCount>1</forkCount>
-                        <reuseForks>false</reuseForks>
                         <excludes>
                             <exclude>**/*IntegrateTest.java</exclude>
                         </excludes>