----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34191/ -----------------------------------------------------------
(Updated May 14, 2015, 9:03 p.m.) Review request for drill, Jacques Nadeau and Venki Korukanti. Changes ------- Addressed review changes, and rebased on current master [583ca4a] Repository: drill-git Description ------- [DRILL-3052](https://issues.apache.org/jira/browse/DRILL-3052), [DRILL-3066](https://issues.apache.org/jira/browse/DRILL-3066): FragmentExecutor must cleanup exactly once Cleanup includes, in order: 1) closing out resources, 2) updating to the correct terminal state, and 3) sending the state to QueryManager exactly once In DRILL-3053 scenario, sendFinalState() is never called In DRILL-3066 scenario, closeOutResources() is called twice Diffs (updated) ----- exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentExecutor.java 8c49d68 exec/java-exec/src/test/java/org/apache/drill/exec/server/TestDrillbitResilience.java f95fbe1 Diff: https://reviews.apache.org/r/34191/diff/ Testing ------- Passes all unit test and regression tests. Started another build. Thanks, Sudheesh Katkam
