[GitHub] [logging-log4j2] jvz commented on a diff in pull request #916: [LOG4J2-2921] Parallel execution for JUnit5 tests

[GitBox <gi...@apache.org>]

jvz commented on code in PR #916:
URL: https://github.com/apache/logging-log4j2/pull/916#discussion_r889732979


##########
log4j-core/pom.xml:
##########
@@ -444,6 +444,30 @@
             <org.apache.activemq.SERIALIZABLE_PACKAGES>*</org.apache.activemq.SERIALIZABLE_PACKAGES>
           </systemPropertyVariables>
         </configuration>
+        <executions>
+            <execution>
+                <id>default-test</id>
+                <phase>test</phase>
+                <goals>
+                  <goal>test</goal>
+                </goals>
+                <configuration>
+                   <includeJUnit5Engines>junit-jupiter</includeJUnit5Engines>
+                   <forkCount>1C</forkCount>
+                   <runOrder>random</runOrder>

Review Comment:
   I've taken to using `<runOrder>balanced</runOrder>` when parallelizing tests. I did set them to random recently in master because I was sick of seeing the same slow tests at the beginning and wanted to change things up. :)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org