CODEWITHSUNDEEP
javawindowsmavencmdprocessbuilder
user8210794
user8210794

Reputation:

Maven exec with ProcessBuilder stops processing

I am running my code as;

mvn exec:java@xxxxxxxxxxxx -Dexec.args="hello"

I don't know why but somehow it stops processing. It is getting stuck after 10 or 20 seconds. But this is happening with the ProcessBuilder;

Here is how I run it:

public static long runCommandLine(String commandLineArgs){
        try {
            ProcessBuilder pb = new ProcessBuilder("cmd.exe", "/c", commandLineArgs);
            Process process = pb.start();
            if(!commandLineArgs.contains("runner")) {
                BufferedReader reader = new BufferedReader(new InputStreamReader(process.getInputStream()));
                while ((reader.readLine()) != null) { /*haha*/}
                process.waitFor();
            }
            return process.pid();
        } catch (Exception e) {
            e.printStackTrace();
            return 0;
        }
    }

I send the .bat file's path to the method as an argument (String commandLineArgs);

.bat file contains lines:

cd C:/xxxx/%username%/xxx
mvn exec:java@xxxxxxxxxxxx -Dexec.args="hello"

If I run the .bat file manually, everything is okay. But with the ProcessBuilder it will get stuck after a while.

pom.xml configutarion:

<plugin>
                    <groupId>org.codehaus.mojo</groupId>
                    <artifactId>exec-maven-plugin</artifactId>
                    <version>1.6.0</version>
                    <executions>
                        <execution>
                            <id>xxxxxxxx</id>
                            <configuration>
                                <mainClass>xxxxxxxxxxx</mainClass>
                            </configuration>
                        </execution>
                   </executions>
</plugin>

Upvotes: 0

Views: 337

Answers (2)

DuncG
DuncG

Reputation: 15186

The cause of a process apparently pausing some time after ProcessBuilder.start() is often related to your application not consuming the stdout and stderr streams as they are generated. So if not calling redirectError/redirectOutput(File) as suggested, just set up a Runnable/Thread to consume both of getInputStream / getErrorStream.

Upvotes: 0

user8210794
user8210794

Reputation:

as khmarbaise mentioned "ProcessBuilder.redirectError and ProcessBuilder.redirectOutput should be used and afterwards you can read from it...", the solution is adding redirects.

Upvotes: 0

Related Questions