-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34603/
-----------------------------------------------------------
(Updated June 30, 2015, 10:37 p.m.)
Review request for drill, Jacques Nadeau, Jason Altekruse, and Sudheesh Katkam.
Bugs: DRILL-3167
https://issues.apache.org/jira/browse/DRILL-3167
Repository: drill-git
Description
-------
- In case of a failure the Foreman will cancel all fragments and move to a FAILING state until
all fragments are terminated
- QueryManager.cancelExecutingFragments() returns false if no fragment available
- Web UI still displays FAILED when Foreman state is FAILING
Diffs
-----
exec/java-exec/src/main/java/org/apache/drill/exec/rpc/control/WorkEventBus.java 3e461ef
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/ProfileResources.java
6656bf6
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/ProfileWrapper.java
dd26a76
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/resources/rest/profile/list.ftl cf92ede
exec/java-exec/src/main/resources/rest/profile/profile.ftl 46cdc83
protocol/src/main/java/org/apache/drill/exec/proto/UserBitShared.java e76d748
protocol/src/main/java/org/apache/drill/exec/proto/beans/QueryResult.java 474e330
protocol/src/main/protobuf/UserBitShared.proto 0451fd2
Diff: https://reviews.apache.org/r/34603/diff/
Testing (updated)
-------
unit tests are passing along with functional and tpch100
Thanks,
abdelhakim deneche
|