----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34008/#review83221 -----------------------------------------------------------
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java <https://reviews.apache.org/r/34008/#comment134128> We should remove the fragment manager from the work bus if it's fragment executor has already been cancelled. Otherwise, it will stay in the work bus forever. - abdelhakim deneche On May 10, 2015, 7:13 p.m., Sudheesh Katkam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34008/ > ----------------------------------------------------------- > > (Updated May 10, 2015, 7:13 p.m.) > > > Review request for drill, abdelhakim deneche, Jacques Nadeau, and Venki > Korukanti. > > > Repository: drill-git > > > Description > ------- > > + Undid the initial patch > + Only swap the implementations > + Rebased on master [87051d4] > > > Diffs > ----- > > exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java > 3e4f3d1 > > Diff: https://reviews.apache.org/r/34008/diff/ > > > Testing > ------- > > Passes all unit tests and regression tests. > > > Thanks, > > Sudheesh Katkam > >
