> On May 10, 2015, 7:55 p.m., abdelhakim deneche wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java, > > line 181 > > <https://reviews.apache.org/r/34008/diff/2/?file=955018#file955018line181> > > > > should we make this line a warning ?
Yes; I will make this change as part of DRILL-2978 (I was working on both patches together but I forgot to make the change as part of this patch). - Sudheesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34008/#review83189 ----------------------------------------------------------- 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 > >
