----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3200/#review3920 -----------------------------------------------------------
Ship it! +1 - Mohammad On 2011-12-14 19:24:03, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3200/ > ----------------------------------------------------------- > > (Updated 2011-12-14 19:24:03) > > > Review request for oozie. > > > Summary > ------- > > Changing the logic so the thread run() that consumes the STD/ERR stream > closes the stream on completion, thus not having a race condition on the > stream closure > > > This addresses bug OOZIE-635. > https://issues.apache.org/jira/browse/OOZIE-635 > > > Diffs > ----- > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/ShellMain.java > 1214413 > > Diff: https://reviews.apache.org/r/3200/diff > > > Testing > ------- > > related testcases pass > > > Thanks, > > Alejandro > >
