----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29907/#review68311 -----------------------------------------------------------
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/QueryManager.java <https://reviews.apache.org/r/29907/#comment112476> QueryManager should communicate to Foreman through the stateListener abstraction. It shouldn't hold a direct reference. exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentExecutor.java <https://reviews.apache.org/r/29907/#comment112478> ? exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentExecutor.java <https://reviews.apache.org/r/29907/#comment112494> This error message doesn't make sense in the method. exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentExecutor.java <https://reviews.apache.org/r/29907/#comment112479> Why runtimeexception? Should be sublcass of drill exception - Jacques Nadeau On Jan. 15, 2015, 12:47 a.m., Hanifi Gunes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29907/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2015, 12:47 a.m.) > > > Review request for drill, Jacques Nadeau, Parth Chandra, and Steven Phillips. > > > Bugs: DRILL-2004 > https://issues.apache.org/jira/browse/DRILL-2004 > > > Repository: drill-git > > > Description > ------- > > DRILL-2004: Foreman should account for early termination of fragments > > > Diffs > ----- > > > exec/java-exec/src/main/java/org/apache/drill/exec/rpc/control/WorkEventBus.java > b9f0a26 > > exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java > 5efc9fa > > exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/QueryManager.java > d4c87d4 > > exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentExecutor.java > 27038d3 > > Diff: https://reviews.apache.org/r/29907/diff/ > > > Testing > ------- > > all tests pass > > > Thanks, > > Hanifi Gunes > >
