----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36209/#review90729 -----------------------------------------------------------
Ship it! Ship It! - Venki Korukanti On July 6, 2015, 10:13 a.m., Sudheesh Katkam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36209/ > ----------------------------------------------------------- > > (Updated July 6, 2015, 10:13 a.m.) > > > Review request for drill and Venki Korukanti. > > > Bugs: DRILL-3450 > https://issues.apache.org/jira/browse/DRILL-3450 > > > Repository: drill-git > > > Description > ------- > > DRILL-3450: Moved methods from AbstractStatusReporter and > NonRootStatusReporter to FragmentStatusReporter > > + Removed StatusReporter interface > + Refactored FragmentStatusReporter > > > Diffs > ----- > > > exec/java-exec/src/main/java/org/apache/drill/exec/work/batch/ControlMessageHandler.java > 9f302a2 > > exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java > 716fb66 > > exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/QueryManager.java > 9318233 > > exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/AbstractStatusReporter.java > 8a40f1b > > exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentExecutor.java > 3409587 > > exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentStatusReporter.java > PRE-CREATION > > exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/NonRootFragmentManager.java > 77440c5 > > exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/NonRootStatusReporter.java > 71da12b > > exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/StatusReporter.java > 424e7fb > > Diff: https://reviews.apache.org/r/36209/diff/ > > > Testing > ------- > > will update > > > Thanks, > > Sudheesh Katkam > >
