2016-01-12 167 views
0

我正在使用Java Executors.newWorkStealingPool();并行运行多个命令。由于其中一些任务预计会超时,所以我为它们添加了2秒的超时时间。进程不终止

ProcessBuilder pb = new ProcessBuilder(cmd.split(" ")); 
Process p = null; 
StringBuilder result = new StringBuilder(); 
try { 
    p = pb.start(); 
    p.waitFor(2000, TimeUnit.MILLISECONDS); 
    BufferedReader errreader = 
    new BufferedReader(new InputStreamReader(p.getErrorStream())); 
    BufferedReader outreader = new BufferedReader(new InputStreamReader(p.getInputStream())); 
    String line = "";   
    while ((line = outreader.readLine())!= null) 
    result.append(line + "\n"); 
    while ((line = errreader.readLine())!= null) 
     result.append(line + "\n"); 
    outreader.close(); 
    errreader.close(); 
    } catch (InterruptedException e) { 
     result.append("INTERRUPTED"); 
    } catch (IOException e) { 
     e.printStackTrace(); 
     System.exit(-1); 
    } 
    finally { 
     p.destroy(); 
    } 
    .... 
    return b; // b is boolean 

提交任务看起来像这样

List<ImmutablePair> curJobs = new ArrayList<ImmutablePair>(); 
for (int i = 0; i < allMutants.size(); i++) { 
    String mutant = allMutants.get(i); 
    t = executor.submit(new mutTask(mutant)); 
    ImmutablePair pair = new ImmutablePair(mutant, t); 
    curJobs.add(pair); 
    } 

最后。我在下面的循环中收集结果。

// data collection loop 
for (int i = 0; i< curJobs.size(); i++) { 
     ImmutablePair p = curJobs.get(i); 
     String m= (String) p.getLeft(); 
    System.out.println(m); 
     t = (Future<Boolean>) p.getRight(); 
    try { 
     boolean res; 
     try { 
      res = t.get(2000L, TimeUnit.MILLISECONDS); 

     } catch (TimeoutException e) { 
      res = true; 
     } 
     finally { 
      t.cancel(true); 
     }} 

问题是,超时似乎并没有终止程序中定义的某些命令或进程。几分钟后数据收集循环变得非常缓慢。 ps的结果显示了一些超出指定超时运行并保持CPU繁忙的作业。我的配置是CentOS上的JDK8。

+1

所以输出,从JavaDoc中,我们知道*“造成当前线程等待,如果必要的话,直到由该Process对象表示的子进程已经终止,或经过指定的等待时间。“*,这意味着此方法将在进程存在或指定时间过去时返回,这意味着进程可能仍在运行。接下来,您将读取流程输出的内容,这意味着如果流程仍然处于崩溃状态,则它仍在处理输出或阻塞......通常我使用子线程“Thread”来读取输入,当waitFor '返回,停止这些线程 – MadProgrammer

+0

...然后销毁进程 – MadProgrammer

+0

谢谢。正如你注意到的那样,问题是没有检查'p.waitFor(2000,TimeUnit.MILLISECONDS);'的输出。 – Amin

回答

0

正如@MadProgrammer指出的问题是读取/ err缓冲区,以防进程终止。为了解决这个问题,你需要检查的p.waitFor(2000, TimeUnit.MILLISECONDS);

try { 
     p = pb.start(); 
     boolean timeout = !p.waitFor(2000, TimeUnit.MILLISECONDS); 
     if (!timeout){ 
      BufferedReader errreader = 
        new BufferedReader(new InputStreamReader(p.getErrorStream())); 
      BufferedReader outreader = 
        new BufferedReader(new InputStreamReader(p.getInputStream())); 

      String line = ""; 
      while ((line = outreader.readLine())!= null) 
       result.append(line + "\n"); 
      while ((line = errreader.readLine())!= null) 
        result.append(line + "\n"); 
      outreader.close(); 
      errreader.close(); 
     } 

    } catch ...