(hadoop) branch trunk updated: HADOOP-19011. Possible ConcurrentModificationException if Exec command fails (#6353)

[adoroszlai]

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

adoroszlai pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/hadoop.git


The following commit(s) were added to refs/heads/trunk by this push:
     new cf21f35526fc HADOOP-19011. Possible ConcurrentModificationException if 
Exec command fails (#6353)
cf21f35526fc is described below

commit cf21f35526fca2d6c2c0518ba0a5d0dc2057f8ad
Author: Doroszlai, Attila <6454655+adorosz...@users.noreply.github.com>
AuthorDate: Thu Dec 14 19:46:19 2023 +0100

    HADOOP-19011. Possible ConcurrentModificationException if Exec command 
fails (#6353)
---
 .../src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java       | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java
 
b/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java
index 86e8d9c2a0ed..a55041f82fbd 100644
--- 
a/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java
+++ 
b/hadoop-maven-plugins/src/main/java/org/apache/hadoop/maven/plugin/util/Exec.java
@@ -72,14 +72,14 @@ public class Exec {
       stdOut.start();
       stdErr.start();
       retCode = p.waitFor();
+      stdOut.join();
+      stdErr.join();
       if (retCode != 0) {
         mojo.getLog().warn(command + " failed with error code " + retCode);
         for (String s : stdErr.getOutput()) {
           mojo.getLog().debug(s);
         }
       }
-      stdOut.join();
-      stdErr.join();
       output.addAll(stdOut.getOutput());
       if (errors != null) {
         errors.addAll(stdErr.getOutput());


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-commits-h...@hadoop.apache.org