This is an automated email from the ASF dual-hosted git repository.
adoroszlai pushed a commit to branch branch-3.3
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/branch-3.3 by this push:
new 284288eeb4a7 HADOOP-19011. Possible ConcurrentModificationException if
Exec command fails (#6358)
284288eeb4a7 is described below
commit 284288eeb4a78bb8c68f440d79060d1eac2cd79a
Author: Doroszlai, Attila <6454655+adorosz...@users.noreply.github.com>
AuthorDate: Fri Dec 15 16:39:25 2023 +0100
HADOOP-19011. Possible ConcurrentModificationException if Exec command
fails (#6358)
(cherry picked from commit cf21f35526fca2d6c2c0518ba0a5d0dc2057f8ad)
---
.../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 528163103da4..50ad6a038577 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
