> On May 11, 2015, 12:45 p.m., abdelhakim deneche wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java, > > line 292 > > <https://reviews.apache.org/r/34008/diff/2/?file=955018#file955018line292> > > > > 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.
I will make this change as part of DRILL-2978. - Sudheesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34008/#review83221 ----------------------------------------------------------- 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 > >
