You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ti...@apache.org on 2019/12/27 18:35:57 UTC

[maven-surefire] branch cli updated: [SUREFIRE-1516] Poor performance in reuseForks=false

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

tibordigana pushed a commit to branch cli
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git


The following commit(s) were added to refs/heads/cli by this push:
     new 1400849  [SUREFIRE-1516] Poor performance in reuseForks=false
1400849 is described below

commit 14008494575316e7c6de428073913ca25cdda39c
Author: tibordigana <ti...@apache.org>
AuthorDate: Fri Dec 27 19:35:50 2019 +0100

    [SUREFIRE-1516] Poor performance in reuseForks=false
---
 .../main/java/org/apache/maven/surefire/booter/CommandReader.java   | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/surefire-api/src/main/java/org/apache/maven/surefire/booter/CommandReader.java b/surefire-api/src/main/java/org/apache/maven/surefire/booter/CommandReader.java
index b71aec0..2e735d9 100644
--- a/surefire-api/src/main/java/org/apache/maven/surefire/booter/CommandReader.java
+++ b/surefire-api/src/main/java/org/apache/maven/surefire/booter/CommandReader.java
@@ -412,6 +412,12 @@ public final class CommandReader
                                 CommandReader.this.wakeupIterator();
                                 callListeners( command );
                                 break;
+                            case BYE_ACK:
+                                callListeners( command );
+                                // After SHUTDOWN no more commands can come.
+                                // Hence, do NOT go back to blocking in I/O.
+                                CommandReader.this.state.set( TERMINATED );
+                                break;
                             default:
                                 callListeners( command );
                                 break;